Skip to content

refactor(operations): move CLI text formatting into descriptors - #58

Merged
bntvllnt merged 1 commit into
mainfrom
feat/2-5-0-operation-formatters
Jul 1, 2026
Merged

refactor(operations): move CLI text formatting into descriptors#58
bntvllnt merged 1 commit into
mainfrom
feat/2-5-0-operation-formatters

Conversation

@bntvllnt

@bntvllnt bntvllnt commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • move CLI human-text formatting into descriptor-backed operation formatters
  • keep CLI JSON output and MCP success/error envelopes unchanged
  • add chained CLI text parity coverage for every registered operation
  • update architecture docs and 2.5.0 roadmap status

Roadmap

  • 2.5.0 / P1 Operation Registry: formatter migration completed

Review

  • Code-review artifact: /tmp/code-review-operation-formatters.json
  • Verdict: CLEAN
  • Hunks: 53/53 reviewed
  • Relevant rules: 12/12 checked

Validation

  • pnpm exec eslint src/operations/formatters.ts src/operations/index.ts src/cli.ts src/mcp/index.ts tests/operation-registry.test.ts tests/operation-registry.e2e.test.ts
  • pnpm typecheck
  • pnpm build
  • pnpm exec vitest run --no-cache --pool threads --no-file-parallelism --maxWorkers=1 tests/operation-registry.test.ts tests/operation-registry.e2e.test.ts
  • pnpm test
  • pnpm verify:cli-real
  • git diff --cached --check
  • node dist/cli.js changes . --json --force
  • node dist/cli.js dependents . src/operations/formatters.ts --json --force
  • node dist/cli.js dependents . src/operations/index.ts --json --force
  • node dist/cli.js hotspots . --metric complexity --limit 8 --json --force
  • local dogfood: node dist/cli.js overview tests/fixture-codebase/src --force
  • local dogfood: node dist/cli.js hotspots tests/fixture-codebase/src --metric coupling --limit 3 --force

@bntvllnt bntvllnt self-assigned this Jul 1, 2026
@bntvllnt bntvllnt added javascript Pull requests that update javascript code documentation Improvements or additions to documentation labels Jul 1, 2026
@bntvllnt
bntvllnt merged commit ae3efeb into main Jul 1, 2026
2 checks passed
@bntvllnt
bntvllnt deleted the feat/2-5-0-operation-formatters branch July 1, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant