Skip to content

PR_26180_OWNER_022-delete-remaining-root-src-files#302

Draft
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_021-archive-legacy-games-samples-teardownfrom
PR_26180_OWNER_022-delete-remaining-root-src-files
Draft

PR_26180_OWNER_022-delete-remaining-root-src-files#302
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_021-archive-legacy-games-samples-teardownfrom
PR_26180_OWNER_022-delete-remaining-root-src-files

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Outcome

HARD STOP. No root src/ files were deleted, archived, or moved.

Reason

The required reference audit found active current validation/test/tooling references that still require root src/shared files. The task explicitly requires a hard stop if runtime, test, package, or CI references still require root src.

Evidence

  • git ls-files -- src before: 65 tracked files
  • git ls-files -- src after: 65 tracked files
  • Targeted root src scan found active references from dev tests, validation scripts, and toolbox-dev guard baselines.
  • InMemoryProjectDataStore.test.mjs: PASS
  • ToolManifestBoundary.test.mjs: PASS
  • ContractIndexValidation.test.mjs: FAILS on missing legacy src/shared/contracts/replayContracts.js

Validation

  • git diff --check: PASS
  • git diff --cached --check: PASS before commit
  • npm run validate:canonical-structure: PASS
  • Targeted root src reference scan: HARD STOP condition confirmed

Reports and Artifact

  • dev/reports/PR_26180_OWNER_022-delete-remaining-root-src-files_report.md
  • dev/reports/PR_26180_OWNER_022-delete-remaining-root-src-files_requirement-checklist.md
  • dev/reports/PR_26180_OWNER_022-delete-remaining-root-src-files_validation-report.md
  • dev/workspace/zips/PR_26180_OWNER_022-delete-remaining-root-src-files_HARD_STOP.zip

Recommendation

Create a follow-up migration PR to move or retire the active tests/scripts/tooling references away from root src/shared, then retry deletion only after root src has no active runtime/test/package/CI dependencies.

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.

2 participants