Repository navigation
Correct source-aware DLP admission and add private corpus audits - #283
Merged
Merged
Conversation
xeonvs
marked this pull request as ready for review
October 30, 2026 15:05
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Merged
xeonvs
added a commit
that referenced
this pull request
Oct 30, 2026
Release the source-aware DLP corrections and local private audit CLI from #283 as v0.14.3. - Stable marker: 0.14.3; next development line: 0.15.0. Update the authorized source epoch, issue metadata, example pin, Towncrier changelog and release notes. - Tracked issue: #282. The Release workflow owns closure after immutable receipt verification. - Archive the full reviewed plan and reset PLANS.md in this PR. Required stable delivery, independent external readback and verified owned-source cleanup remain outside the closed repository plan. Validation: protected feature CI passed the supported Linux/macOS Python matrix; development TestPyPI workflow 41079397775 verified published artifacts, provenance and installs. The final runtime quality gate passed 3747 tests and 357 subtests with 88.79% coverage. Own-tool review completed 51/51 files with zero findings and an admitted diagram in #283; both complete post-quality corpus replays were unchanged. All 156 focused release/environment/documentation checks and lifecycle/instruction guards pass. One aggregate Codex Security scan was completed for the implementation; release preparation changes documentation and release metadata. Publication and external reconciliation are pending this PR's protected merge. No later plan-closure PR is needed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Correct DLP admission for proven nonsecret source expressions, distinguish completed analysis from filtered publication, and add the local
ocr-ci dlp-auditdiagnostic command. Related issue: #282; stable receipt verification remains the issue-closure owner.Validation: the full sequential quality gate passes 3747 tests and 357 subtests (47 conditional skips), with 88.79% coverage and every scoped floor. Clean wheel/sdist installation checks pass. Both complete captured corpora replay unchanged after acceptance; unexplained transitions and lost confirmed protection are zero, while untriaged observations remain explicit. Extended DLP/source qualification passes 545 tests/properties at 1000 examples per property; 24 official native grouping/round/diagram checks pass. One aggregate Security scan has no findings; subsequent corrections have focused independent review. Actual ChatGPT/Astra high self-review of
72abb603..b4cfbb1completed 51/51 selected files with zero raw or delivered findings and an admitted diagram.DLP admission and private audit flow
flowchart LR n0["Offline audit#58; select files#44; Git blobs#44; stdin or snapshot replay"] n1["Bound input capture and bind corpus to private authenticated state"] n2["Remove owned raw snapshots while preserving originals and evidence"] n3["Annotate exact hits and compare complete identical populations"] n4["Request owner-group diagrams and require conditional-edge references"] n5["Apply sink-aware DLP with independent secret and contact checks"] n6["Raise result#47;preflight to 85#37;#59; add local per-module 90#37; floors"] n7["Separate complete analysis from incomplete output in review summaries"] n8["Project safe findings and omit unsafe optional code fields"] n9["Prove exact source ranges and cache bounded syntax interpretation"] n10["Persist bounded audit hits#44; completeness and scanner identity"] n11["Isolated worker evaluates whole-file and nonempty-line units"] n0 --> n1 n1 --> n11 n3 -->|"Operator requests cleanup"| n2 n4 -->|"Validate diagram labels with their dedicated sink"| n5 n5 --> n8 n8 -->|"Review output is admission-filtered or publication-filtered"| n7 n8 -->|"Audit checks code admission against publication projection"| n10 n9 --> n5 n10 --> n3 n11 --> n9