farhoodlabs/paperclip
8
0
Fork 2
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(adapters): add capability flags to ServerAdapterModule #2

Closed
farhoodliquor-paperclip[bot] wants to merge 2 commits from feat/adapter-plugin-capabilities into master
pull from: feat/adapter-plugin-capabilities
merge into: farhoodlabs:master
Conversation 0 Commits 2 Files Changed 12
+24918 -4076
farhoodliquor-paperclip[bot] commented 2026-04-13 01:37:55 +00:00 (Migrated from github.com)
Copy Link
Copy Source

Summary

Replace 4 hardcoded adapter type lists across the server and UI with declarative capability flags on ServerAdapterModule, enabling external adapter plugins to declare their capabilities without modifying Paperclip source.

  • Add supportsInstructionsBundle, instructionsPathKey, and requiresMaterializedRuntimeSkills optional fields to ServerAdapterModule
  • Set explicit flags on all 10 built-in adapters in registry.ts
  • Replace hardcoded lists in agents.ts with capability-aware helpers (with legacy fallbacks)
  • Expose capabilities object in GET /api/adapters response
  • New useAdapterCapabilities React hook for UI components
  • Replace hardcoded allowlists/denylists in AgentDetail.tsx, AgentConfigForm.tsx, OnboardingWizard.tsx
  • 6 new tests (adapter-registry + adapter-routes), all passing
  • New "Capability Flags" documentation section in docs/adapters/creating-an-adapter.md
  • Fixes latent bug: gemini_local was missing from AgentDetail.tsx isLocal allowlist

Test plan

  • adapter-registry.test.ts: capability flags exposed, undefined when unset, correct on built-ins
  • adapter-routes.test.ts: capabilities in API response, correct per adapter, derived supportsSkills
  • Manual: verify instructions bundle tab shows for local adapters, hidden for non-local
  • Manual: verify agent config form shows cwd/instructions/skills for local adapters only
  • Manual: verify external adapter with supportsInstructionsBundle: true gets bundle editor

🤖 Generated with Claude Code

## Summary Replace 4 hardcoded adapter type lists across the server and UI with declarative capability flags on `ServerAdapterModule`, enabling external adapter plugins to declare their capabilities without modifying Paperclip source. - Add `supportsInstructionsBundle`, `instructionsPathKey`, and `requiresMaterializedRuntimeSkills` optional fields to `ServerAdapterModule` - Set explicit flags on all 10 built-in adapters in `registry.ts` - Replace hardcoded lists in `agents.ts` with capability-aware helpers (with legacy fallbacks) - Expose `capabilities` object in `GET /api/adapters` response - New `useAdapterCapabilities` React hook for UI components - Replace hardcoded allowlists/denylists in `AgentDetail.tsx`, `AgentConfigForm.tsx`, `OnboardingWizard.tsx` - 6 new tests (adapter-registry + adapter-routes), all passing - New "Capability Flags" documentation section in `docs/adapters/creating-an-adapter.md` - Fixes latent bug: `gemini_local` was missing from `AgentDetail.tsx` isLocal allowlist ## Test plan - [x] `adapter-registry.test.ts`: capability flags exposed, undefined when unset, correct on built-ins - [x] `adapter-routes.test.ts`: capabilities in API response, correct per adapter, derived supportsSkills - [ ] Manual: verify instructions bundle tab shows for local adapters, hidden for non-local - [ ] Manual: verify agent config form shows cwd/instructions/skills for local adapters only - [ ] Manual: verify external adapter with `supportsInstructionsBundle: true` gets bundle editor 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
fl_blake (Blake MIddleware) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) flux (Flux CD) admin (Gitea Admin) fl_hugh (Hugh Commit) renovate (Mend Renovate) fl_warren (Warren Bufferpool)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: farhoodlabs/paperclip#2
Delete Branch "feat/adapter-plugin-capabilities"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?