charles/claude-hooks
1
0
Fork 0
Code Issues 10 Pull requests Projects Releases Packages 1 Wiki Activity Actions

feat: SR-9 skill preview endpoint + SkillsTab modal (#877) #892

Merged
charles merged 1 commit from feat/877-skills-preview-sr9 into main 2026-05-05 23:28:53 +00:00
Conversation 0 Commits 1 Files changed 8 +739 -7
claude-desktop commented 2026-05-05 23:21:34 +00:00
Collaborator
Copy link

Implements issue #877 (SR-9).

Summary

  • POST /agents/:agentType/skills/preview — body { name, vars?, instance_id? }. Response { rendered, applied: { caveman, artifact_style, prompt_appendix } }. Uses renderPrompt with filesystemFallback: false. Stub vars default to { issue_number: 0, repo: "<preview>", title: "<preview>" }. Missing skill returns 404 with the same actionable hint as dispatch.
  • resolveRepresentativeAgentForType / resolveAgentByRowId for merged ResolvedAgent context.
  • resolveSkillPreviewApplied in skill-loader for applied flags.
  • SkillsTab: Preview button per row; modal with monospace body, applied flags, Copy.
  • Tests: preview-endpoint.test.ts, skills-tab.preview.test.tsx.

Closes #877 when merged.

Implements [issue #877](https://forge.jacquin.app/charles/claude-hooks/issues/877) (SR-9). ## Summary - POST `/agents/:agentType/skills/preview` — body `{ name, vars?, instance_id? }`. Response `{ rendered, applied: { caveman, artifact_style, prompt_appendix } }`. Uses `renderPrompt` with `filesystemFallback: false`. Stub vars default to `{ issue_number: 0, repo: "<preview>", title: "<preview>" }`. Missing skill returns 404 with the same actionable hint as dispatch. - `resolveRepresentativeAgentForType` / `resolveAgentByRowId` for merged ResolvedAgent context. - `resolveSkillPreviewApplied` in skill-loader for applied flags. - SkillsTab: Preview button per row; modal with monospace body, applied flags, Copy. - Tests: preview-endpoint.test.ts, skills-tab.preview.test.tsx. Closes #877 when merged.
feat: skill preview endpoint and SkillsTab modal (SR-9 #877)
All checks were successful
qa / sql-layer-check (pull_request) Successful in 10s
Details
qa / dockerfile (pull_request) Successful in 10s
Details
qa / db-schema (pull_request) Successful in 11s
Details
qa / qa-1 (pull_request) Successful in 50s
Details
qa / qa (pull_request) Successful in 0s
Details
1eadf39367
 
Add POST /agents/:type/skills/preview using renderPrompt plus applied-flag metadata,
resolveRepresentativeAgentForType for type-anchor previews, and a monospace modal with copy.

Co-authored-by: Cursor <cursoragent@cursor.com>
charles deleted branch feat/877-skills-preview-sr9 2026-05-05 23:28:53 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
area:agents
Agent types, pool scheduling, per-instance config

  
area:dashboard
Dashboard UI and observability surfaces

  
area:database
DB layer — schema, migrations, ORM, raw SQL

  
area:design
UI/UX mockup work — routes to designer agent

  
area:design-review
Design review dispatch — routes to design-reviewer agent

  
area:flows
Flow runner — YAML loader, executor, op registry, expression eval

  
area:infra
Deployment, isolation, containers, systemd units

  
area:meta
Tracking, scaffolding, project setup

  
area:security
Security — routes to reviewer-security (opus)

  
area:sessions
Session-id store, Claude SDK resume logic

  
area:webhook
Forgejo webhook routing and handlers

  
area:workdir
Clone cache, worktrees, git identity

  
security
Security-sensitive issue

  
type:bug
Bug

  
type:chore
Chore

  
type:meta
Tracking or decisions, not implementation work

  
type:user-story
User story

No labels
area:agents
area:dashboard
area:database
area:design
area:design-review
area:flows
area:infra
area:meta
area:security
area:sessions
area:webhook
area:workdir
security
type:bug
type:chore
type:meta
type:user-story
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
charles/claude-hooks!892
No description provided.