Skip to content

feat: planner scope management CLI commands#11

Open
alexzoo wants to merge 1 commit intoOzeroHAX:mainfrom
alexzoo:feat/planner-scope
Open

feat: planner scope management CLI commands#11
alexzoo wants to merge 1 commit intoOzeroHAX:mainfrom
alexzoo:feat/planner-scope

Conversation

@alexzoo
Copy link
Copy Markdown

@alexzoo alexzoo commented Feb 26, 2026

Summary

  • Add CLI subcommands: assistagents planner-scope list|add|remove|reset
  • Allow post-install management of planner's .md edit permissions without reinstall
  • Add anchors in planner.md for in-place permission updates
  • Add slash-command /planner-scope for build/dev agent

Changes

New files

  • src/planner-scope.ts — core logic for scope management
  • templates/commands/planner-scope.md — slash-command template

Modified files

  • src/cli.ts — CLI routing for planner-scope subcommands
  • templates/agents/build/planner.md — anchors for managed permissions

Tests

  • tests/planner-scope.test.ts — unit tests (14 tests)
  • tests/planner-scope-cli.test.ts — integration tests (11 tests)
  • tests/agent-placeholders.test.ts — anchor verification (1 test)

Validation rules

Pattern Without --force With --force
docs/**.md ✅ Allowed ✅ Allowed
**/*.md ❌ Blocked (wide) ✅ Allowed + warning
/tmp/x.md ❌ Blocked (absolute) ❌ Blocked (always)
../x.md ❌ Blocked (..) ❌ Blocked (always)

Test plan

npm run typecheck  # OK
npm test           # 41/42 PASS (1 unrelated fail)
npm run build      # OK

Documentation

  • Plan: ai-docs/dev-plans/2026-02-26-planner-scope-management.md
  • Test report: ai-docs/reports/test-reports/2026-02-26-planner-scope-management.md
  • Code review: ai-docs/reports/2026-02-26-planner-scope-code-review.md

- Add CLI subcommands: planner-scope list|add|remove|reset
- Add anchors in planner.md for in-place permission updates
- Validate patterns: block absolute paths, .., and wide patterns
- Add slash-command /planner-scope for build/dev agent
- Update write_scope rule to reference configurable patterns
- Add unit and integration tests (41 tests)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant