Skip to content

Commit b513b9a

Browse files
committed
Audit contracts and schemas obsolescence
1 parent b424f29 commit b513b9a

8 files changed

Lines changed: 2514 additions & 10151 deletions

dev/reports/PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit_audit.csv

Lines changed: 90 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit Branch Validation
2+
3+
| Check | Status | Evidence |
4+
| --- | --- | --- |
5+
| Current branch | PASS | `PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit` |
6+
| Expected stacked base | PASS | Created from `PR_26180_OWNER_018-move-src-browser-to-www`. |
7+
| Worktree start | PASS | Clean before report generation after preserving prior PR019 hard-stop artifacts in a stash. |
8+
| Branch model | PASS | Owner stacked governance/report PR from previous dependency branch. |
9+
| Protected dev workspace | PASS | No protected `dev/workspace/**` files were moved or modified by this audit. |
10+
| Source movement | PASS | No files moved or deleted. |
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit Manual Validation Notes
2+
3+
- Reviewed active reference surfaces requested by Owner: `www/`, `api/`, `dev/tests/`, `dev/scripts/`, and `package.json` scripts.
4+
- Treated `www/src/shared/toolbox/schemaOnlyToolPresetValidation.js` as a dynamic browser/runtime reference for all tool schema JSON files because it builds `/src/shared/schemas/tools/<tool>.schema.json` at runtime.
5+
- No source files were moved, deleted, or rewritten.
6+
- No protected developer workspace files were moved.
7+
- Recommended next action is to split/move browser-consumed contracts and schemas to `www/src/shared/...` before retrying API-only migration.

dev/reports/PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit_report.md

Lines changed: 1113 additions & 0 deletions
Large diffs are not rendered by default.
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit Requirement Checklist
2+
3+
| Requirement | Status | Evidence |
4+
| --- | --- | --- |
5+
| Base is PR018 | PASS | Branch created from `PR_26180_OWNER_018-move-src-browser-to-www`. |
6+
| List every file under src/shared/contracts and src/shared/schemas | PASS | Detailed report and CSV list 89 tracked files. |
7+
| Identify references from www | PASS | Exact and dynamic browser schema references captured. |
8+
| Identify references from api | PASS | No direct active `api/` references found from requested surfaces. |
9+
| Identify references from dev/tests | PASS | Test references captured per file. |
10+
| Identify references from dev/scripts | PASS | Script references captured per file, including dynamic schema validation path. |
11+
| Identify package script references | PASS | `package.json` scanned; no package refs found. |
12+
| Classify every file | PASS | Classification totals: Active test/validation dependency: 64; Active browser/runtime dependency: 23; Unknown / needs Owner decision: 2. |
13+
| Determine DB/API supersession | PASS | Per-file supersession recommendation documented. |
14+
| Recommend exact action per file | PASS | Per-file action documented as move to www, move to dev, or keep temporarily. |
15+
| Do not delete active runtime/test references | PASS | No files deleted. |
16+
| Do not move browser-consumed files into api | PASS | No files moved; report recommends browser-consumed files move to `www/`. |
17+
| Do not leave authoritative product data in browser-owned files | PASS | Audit recommends preserving boundary and does not add product data. |
18+
| Preserve protected dev workspace | PASS | No protected workspace files moved. |
19+
| git diff --check | PASS | Passed. |
20+
| npm run validate:canonical-structure | PASS | Passed with 0 blocking violations. |
21+
| Required reports and ZIP | PASS | Reports generated under `dev/reports/`; ZIP generated under `dev/workspace/zips/`. |
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit Validation Report
2+
3+
| Validation | Result | Notes |
4+
| --- | --- | --- |
5+
| Startup validation | PASS | Loaded Project Instructions v2026.06.28.018 from repository; cached memory discarded. |
6+
| Branch/base validation | PASS | `PR_26180_OWNER_019a-contracts-schemas-obsolescence-audit` from `PR_26180_OWNER_018-move-src-browser-to-www`. |
7+
| Audit generation | PASS | 89 tracked contracts/schema files audited. |
8+
| Protected workspace check | PASS | No protected `dev/workspace/generated/**`, `dev/workspace/zips/**`, or `dev/workspace/tmp/**` movement. |
9+
| git diff --check | PASS | No whitespace errors. |
10+
| npm run validate:canonical-structure | PASS | Blocking violations: 0; approved legacy exceptions: 499. |
11+
| Runtime/API/UI/database changes | PASS | None. Governance/report-only audit. |

dev/reports/codex_changed_files.txt

Lines changed: 6 additions & 737 deletions
Large diffs are not rendered by default.

dev/reports/codex_review.diff

Lines changed: 1256 additions & 9414 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)