Skip to content

feat(devtools): desloppify Multica issue creator + CI docs (APR-217)#57

Open
JacobSampson wants to merge 1 commit into
mainfrom
APR-217/desloppify-issue-creator
Open

feat(devtools): desloppify Multica issue creator + CI docs (APR-217)#57
JacobSampson wants to merge 1 commit into
mainfrom
APR-217/desloppify-issue-creator

Conversation

@JacobSampson

Copy link
Copy Markdown
Contributor

Summary

  • Adds devtools create-issues command that reads a desloppify scan JSON file and creates Multica child issues for T1/T2 findings, grouped by detector per package
  • Adds src/multica/issue-creator.ts with injectable executor for clean testability in ESM
  • Adds CI integration guide to README covering threshold gating and optional Multica issue creation on merge to main
  • 12 new tests covering grouping, filtering, dry-run mode, executor invocation, error handling, and edge cases

Changes

File What
src/multica/issue-creator.ts Core issue-creator logic with CreateIssuesOptions.executor injection
src/commands/create-issues.ts devtools create-issues CLI command
src/cli.ts Wires makeCreateIssuesCommand() into the CLI
src/__tests__/issue-creator.test.ts 12 tests (28 total, all passing)
README.md devtools create-issues docs + CI integration section

Test plan

  • pnpm test — 28 tests pass, 0 failures
  • pnpm typecheck — 0 errors
  • pnpm lint — 0 errors (console warnings are pre-existing across the package)
  • pnpm build — ESM + CJS + DTS build success
  • Dry-run mode tested: executor never called, correct issue count returned
  • Grouping by detector verified: 2 T1 findings with same detector → 1 issue
  • maxTier=1 filters out T2 findings
  • --parent and --project flags forwarded in exec command
  • Executor errors captured into errors[] without throwing

🤖 Generated with Claude Code

Implements the remaining APR-217 deliverables on top of the existing
desloppify runner: a `create-issues` CLI command that reads scan JSON
output and creates Multica child issues for T1/T2 findings (grouped by
detector per package), plus a CI integration guide in the README.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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