Skip to content

docs: action 2026-06-14 audit batch-1 rulings (B-4/B-5/S-1/S-2/A-1)#217

Merged
lesnik512 merged 1 commit into
mainfrom
fix/audit-doc-rulings-batch1
Jun 14, 2026
Merged

docs: action 2026-06-14 audit batch-1 rulings (B-4/B-5/S-1/S-2/A-1)#217
lesnik512 merged 1 commit into
mainfrom
fix/audit-doc-rulings-batch1

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

Actions the approved batch-1 rulings from the 2026-06-14 deep audit — the doc/test/comment-only findings. No production behavior changes.

ID Ruling Change
B-4 document (done) + pin Test: an override resolves a deeper-scoped provider from a shallower container (intentional, per testing-and-overrides.md)
B-5 document resolution.md: union parameterized-generic degrades to its origin for matching (vs. a bare generic rejected at declaration)
S-1 document validation.md: runtime resolution has no cycle guard (raw RecursionError); use validate() in dev
S-2 document errors-and-exceptions.md: messages may embed wrapped creator/finalizer error text; don't echo raw exceptions to untrusted clients
A-1 accept + comment factory.py comment on the GIL-benign compile-outside-lock; nogil caveat recorded in deferred.md
A-2 close Already documented intentional (2026-06-12 X-1); no action

Bundle (plan-only, spec = the audit report): planning/changes/active/2026-06-14.03-audit-doc-rulings-batch1/.

Test Plan

  • New B-4 pinning test passes; full suite 206 passed, 100% coverage (just test-ci)
  • just lint-ci clean (ruff + ty)
  • mkdocs build --strict OK

🤖 Generated with Claude Code

Doc/test/comment-only, no production behavior change:
- B-4: pin override-bypasses-scope-check-from-shallower-container (test)
- B-5: note union parameterized-generic -> origin degradation (resolution.md)
- S-1: note runtime has no cycle guard; use validate() (validation.md)
- S-2: note wrapped creator/finalizer error text (errors-and-exceptions.md)
- A-1: comment GIL-benign compile-outside-lock; nogil caveat in deferred.md
A-2 closed (already documented intentional).

Spec: planning/audits/2026-06-14-deep-audit-report.md

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lesnik512 lesnik512 merged commit 141f860 into main Jun 14, 2026
7 checks passed
@lesnik512 lesnik512 deleted the fix/audit-doc-rulings-batch1 branch June 14, 2026 12:01
lesnik512 added a commit that referenced this pull request Jun 14, 2026
…217)

Move bundle to archive (status: shipped, pr: 217); update README Index; mark
B-4/B-5/S-1/S-2/A-1/A-2 resolved in the 2026-06-14 audit report Status line.

Co-Authored-By: Claude Opus 4.8 (1M context) <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