Skip to content

docs(async-audit): mark migration plan complete (Phase 4 merged)#233

Merged
pcalnon merged 1 commit intomainfrom
docs/async-route-audit-migration-complete
May 7, 2026
Merged

docs(async-audit): mark migration plan complete (Phase 4 merged)#233
pcalnon merged 1 commit intomainfrom
docs/async-route-audit-migration-complete

Conversation

@pcalnon
Copy link
Copy Markdown
Owner

@pcalnon pcalnon commented May 7, 2026

Summary

Closes out the async-route audit migration. All four phases shipped across the four in-scope repos (juniper-data, juniper-cascor, juniper-canopy, juniper-cascor-worker) on 2026-05-05–06.

Adds a top-level "Migration ledger" section to notes/ASYNC_ROUTE_AUDIT_HOOK_MIGRATION_PLAN.md with:

  • Every PR for each phase × repo (15 PRs total).
  • Outcome against the original §8 success criteria.
  • Deferred follow-ups: branch-protection required-checks (UI-only), optional centralised deny-list, doc-link hygiene.

The rest of the plan is preserved verbatim for the audit trail.

Plan recap

Phase What it did Result
0 Enumerate violations 27 sites across 4 repos; worker clean
1 Wire ruff hook in disabled state (stages: [manual]) 4 PRs merged
2 Flip to soft-fail visibility (--exit-zero + continue-on-error: true) 4 PRs merged; lesson learned (--exit-zero is required to suppress red on the step)
3 Clean up violations data#96 (3 fixes), canopy#247 (5 fixes); cascor & worker no-ops
4 Drop soft-fail flags, hard-fail on new violations 4 PRs merged

Total elapsed: ~36 wall-clock hours, single human in the loop.

Test plan

  • CI green on this docs-only PR
  • Branch-protection rules updated in each repo to require the new Async-route audit (BUG-JD-10 class) check (separate manual step, GitHub UI)

🤖 Generated with Claude Code

All four phases of the async-route audit migration shipped in
~36 wall-clock hours (Phase 0 enumeration through Phase 4 enforce
merges, 2026-05-06). Adds a top-level "Migration ledger" section
listing every PR for each phase × repo, the outcome against §8
success criteria, and the deferred follow-ups (branch-protection
required-checks, optional centralised deny-list).

The rest of the plan is preserved verbatim for the audit trail.

Refs:
- juniper-data PRs #92, #94, #96, #98
- juniper-cascor PRs #231, #232, #235
- juniper-canopy PRs #243, #244, #247, #248
- juniper-cascor-worker PRs #55, #56, #57

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@pcalnon pcalnon self-assigned this May 7, 2026
Copy link
Copy Markdown
Owner Author

@pcalnon pcalnon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@pcalnon pcalnon merged commit 9b4f6a7 into main May 7, 2026
19 checks passed
@pcalnon pcalnon deleted the docs/async-route-audit-migration-complete branch May 7, 2026 07:59
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