Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ The current repository did not contain `docs_build/tool-planning/sprites/`, and
- toolbox/sprites/index.html
- src/shared/toolbox/tool-metadata-inventory.js
- dev/tests/playwright/tools/SpritesToolShell.spec.mjs
- docs_build/dev/reports/PR_26179_CHARLIE_022-sprites-tool-shell.md
- docs_build/dev/reports/codex_changed_files.txt
- docs_build/dev/reports/codex_review.diff
- dev/reports/PR_26179_CHARLIE_022-sprites-tool-shell.md
- dev/reports/codex_changed_files.txt
- dev/reports/codex_review.diff

## Branch Validation

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_022-sprites-tool-shell
Canonical ZIP path: dev/workspace/zip/PR_26179_CHARLIE_023-sprites-toolbar-placeholders_delta.zip
Canonical ZIP path: dev/workspace/zips/PR_26179_CHARLIE_023-sprites-toolbar-placeholders_delta.zip

## Summary

Expand Down Expand Up @@ -65,4 +65,4 @@ Results:

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_023-sprites-toolbar-placeholders_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_023-sprites-toolbar-placeholders_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_023-sprites-toolbar-placeholders
Canonical ZIP path: dev/workspace/zip/PR_26179_CHARLIE_024-sprites-canvas-grid_delta.zip
Canonical ZIP path: dev/workspace/zips/PR_26179_CHARLIE_024-sprites-canvas-grid_delta.zip

## Summary

Expand Down Expand Up @@ -59,4 +59,4 @@ Results:

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_024-sprites-canvas-grid_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_024-sprites-canvas-grid_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_024-sprites-canvas-grid
Canonical ZIP path: dev/workspace/zip/PR_26179_CHARLIE_025-sprites-basic-drawing_delta.zip
Canonical ZIP path: dev/workspace/zips/PR_26179_CHARLIE_025-sprites-basic-drawing_delta.zip

## Summary

Expand Down Expand Up @@ -61,4 +61,4 @@ Results:

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_025-sprites-basic-drawing_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_025-sprites-basic-drawing_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_025-sprites-basic-drawing
Canonical ZIP path: dev/workspace/zip/PR_26179_CHARLIE_026-sprites-palette-panel_delta.zip
Canonical ZIP path: dev/workspace/zips/PR_26179_CHARLIE_026-sprites-palette-panel_delta.zip

## Summary

Expand Down Expand Up @@ -58,4 +58,4 @@ Results:

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_026-sprites-palette-panel_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_026-sprites-palette-panel_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_026-sprites-palette-panel
Canonical ZIP path: dev/workspace/zip/PR_26179_CHARLIE_027-sprites-preview-export_delta.zip
Canonical ZIP path: dev/workspace/zips/PR_26179_CHARLIE_027-sprites-preview-export_delta.zip

## Summary

Expand Down Expand Up @@ -59,4 +59,4 @@ Results:

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_027-sprites-preview-export_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_027-sprites-preview-export_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_027-sprites-preview-export
Canonical ZIP path for this batch: dev/workspace/zip/PR_26179_CHARLIE_028-sprites-editor-polish_delta.zip
Canonical ZIP path for this batch: dev/workspace/zips/PR_26179_CHARLIE_028-sprites-editor-polish_delta.zip

## Summary

Expand Down Expand Up @@ -69,4 +69,4 @@ PR_26179_CHARLIE_029-sprites-clear-reset-controls

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_028-sprites-editor-polish_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_028-sprites-editor-polish_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Team: CHARLIE
Workflow: stacked feature workflow
Base branch: PR_26179_CHARLIE_028-sprites-editor-polish
Canonical ZIP path for this batch: dev/workspace/zip/PR_26179_CHARLIE_029-sprites-clear-reset-controls_delta.zip
Canonical ZIP path for this batch: dev/workspace/zips/PR_26179_CHARLIE_029-sprites-clear-reset-controls_delta.zip

## Summary

Expand Down Expand Up @@ -68,4 +68,4 @@ PR_26179_CHARLIE_030-sprites-undo-redo

## ZIP Path

`dev/workspace/zip/PR_26179_CHARLIE_029-sprites-clear-reset-controls_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_029-sprites-clear-reset-controls_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -70,4 +70,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_030-sprites-undo-redo_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_030-sprites-undo-redo_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -66,4 +66,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_031-sprites-picker-zoom_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_031-sprites-picker-zoom_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -68,4 +68,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_032-sprites-shape-tools_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_032-sprites-shape-tools_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -72,4 +72,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_033-sprites-canvas-preview-sync_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_033-sprites-canvas-preview-sync_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -59,4 +59,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_034-sprites-frame-strip_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_034-sprites-frame-strip_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_035-sprites-frame-editing_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_035-sprites-frame-editing_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -60,4 +60,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_036-sprites-animation-preview_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_036-sprites-animation-preview_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_037-sprites-animation-export_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_037-sprites-animation-export_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -63,4 +63,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_038-sprites-grid-dimension-fix_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_038-sprites-grid-dimension-fix_delta.zip`
Original file line number Diff line number Diff line change
Expand Up @@ -75,4 +75,4 @@ PASS - `npx playwright test dev/tests/playwright/tools/SpritesToolShell.spec.mjs

## ZIP

`dev/workspace/zip/PR_26179_CHARLIE_039-sprites-color-and-zoom-fix_delta.zip`
`dev/workspace/zips/PR_26179_CHARLIE_039-sprites-color-and-zoom-fix_delta.zip`
103 changes: 103 additions & 0 deletions dev/reports/PR_26179_CHARLIE_040-dev-artifact-location-cleanup.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,103 @@
# PR_26179_CHARLIE_040-dev-artifact-location-cleanup

Team: CHARLIE

Base branch: PR_26179_CHARLIE_039-sprites-color-and-zoom-fix

Branch: PR_26179_CHARLIE_040-dev-artifact-location-cleanup

## Scope

Development artifact location cleanup only. No runtime behavior, Sprite Editor functionality, DB/API/schema, or start_of_day files were changed.

## Project Instructions Read

- dev/build/ProjectInstructions/addendums/batch_governance_mode.md
- dev/build/ProjectInstructions/repository/canonical_repository_structure.md
- dev/build/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md
- dev/build/ProjectInstructions/addendums/documentation_ownership.md

## Canonical Artifact Mapping

| Legacy path | Canonical path | Action |
| --- | --- | --- |
| docs_build/dev/reports/PR_26179_CHARLIE_0*.md | dev/reports/PR_26179_CHARLIE_0*.md | Moved generated Charlie Sprite PR reports |
| docs_build/dev/reports/codex_changed_files.txt | dev/reports/codex_changed_files.txt | Removed legacy tracked copy; regenerated canonical report |
| docs_build/dev/reports/codex_review.diff | dev/reports/codex_review.diff | Removed legacy tracked copy; regenerated canonical diff |
| dev/workspace/zip/ | dev/workspace/zips/ | Updated Charlie Sprite report references to canonical ZIP path |

Canonical ZIP path confirmed from current Project Instructions: dev/workspace/zips/

## Files Moved

- docs_build/dev/reports/PR_26179_CHARLIE_022-sprites-tool-shell.md -> dev/reports/PR_26179_CHARLIE_022-sprites-tool-shell.md
- docs_build/dev/reports/PR_26179_CHARLIE_023-sprites-toolbar-placeholders.md -> dev/reports/PR_26179_CHARLIE_023-sprites-toolbar-placeholders.md
- docs_build/dev/reports/PR_26179_CHARLIE_024-sprites-canvas-grid.md -> dev/reports/PR_26179_CHARLIE_024-sprites-canvas-grid.md
- docs_build/dev/reports/PR_26179_CHARLIE_025-sprites-basic-drawing.md -> dev/reports/PR_26179_CHARLIE_025-sprites-basic-drawing.md
- docs_build/dev/reports/PR_26179_CHARLIE_026-sprites-palette-panel.md -> dev/reports/PR_26179_CHARLIE_026-sprites-palette-panel.md
- docs_build/dev/reports/PR_26179_CHARLIE_027-sprites-preview-export.md -> dev/reports/PR_26179_CHARLIE_027-sprites-preview-export.md
- docs_build/dev/reports/PR_26179_CHARLIE_028-sprites-editor-polish.md -> dev/reports/PR_26179_CHARLIE_028-sprites-editor-polish.md
- docs_build/dev/reports/PR_26179_CHARLIE_029-sprites-clear-reset-controls.md -> dev/reports/PR_26179_CHARLIE_029-sprites-clear-reset-controls.md
- docs_build/dev/reports/PR_26179_CHARLIE_030-sprites-undo-redo.md -> dev/reports/PR_26179_CHARLIE_030-sprites-undo-redo.md
- docs_build/dev/reports/PR_26179_CHARLIE_031-sprites-picker-zoom.md -> dev/reports/PR_26179_CHARLIE_031-sprites-picker-zoom.md
- docs_build/dev/reports/PR_26179_CHARLIE_032-sprites-shape-tools.md -> dev/reports/PR_26179_CHARLIE_032-sprites-shape-tools.md
- docs_build/dev/reports/PR_26179_CHARLIE_033-sprites-canvas-preview-sync.md -> dev/reports/PR_26179_CHARLIE_033-sprites-canvas-preview-sync.md
- docs_build/dev/reports/PR_26179_CHARLIE_034-sprites-frame-strip.md -> dev/reports/PR_26179_CHARLIE_034-sprites-frame-strip.md
- docs_build/dev/reports/PR_26179_CHARLIE_035-sprites-frame-editing.md -> dev/reports/PR_26179_CHARLIE_035-sprites-frame-editing.md
- docs_build/dev/reports/PR_26179_CHARLIE_036-sprites-animation-preview.md -> dev/reports/PR_26179_CHARLIE_036-sprites-animation-preview.md
- docs_build/dev/reports/PR_26179_CHARLIE_037-sprites-animation-export.md -> dev/reports/PR_26179_CHARLIE_037-sprites-animation-export.md
- docs_build/dev/reports/PR_26179_CHARLIE_038-sprites-grid-dimension-fix.md -> dev/reports/PR_26179_CHARLIE_038-sprites-grid-dimension-fix.md
- docs_build/dev/reports/PR_26179_CHARLIE_039-sprites-color-and-zoom-fix.md -> dev/reports/PR_26179_CHARLIE_039-sprites-color-and-zoom-fix.md

## References Updated

- Updated moved Charlie Sprite PR reports from docs_build/dev/reports/ to dev/reports/.
- Updated moved Charlie Sprite PR reports from dev/workspace/zip/ to dev/workspace/zips/.
- Left unrelated historical/report artifacts untouched, including docs_build/dev/reports/sprite-history-inventory.md.

## Files Intentionally Not Moved

- Product planning intended to remain in docs_build/
- Architecture documents intended to remain in docs_build/
- DDL, DML, and seed files
- Documentation intended to remain in docs_build/
- Runtime, Sprite Editor, DB/API/schema, and start_of_day files

## Validation Checklist

| Requirement | Result | Notes |
| --- | --- | --- |
| Batch governance mode reviewed | PASS | Read dev/build/ProjectInstructions/addendums/batch_governance_mode.md |
| Canonical artifact paths confirmed | PASS | Reports: dev/reports/; ZIPs: dev/workspace/zips/ |
| Generated Charlie Sprite reports moved out of docs_build/ | PASS | PR 022 through PR 039 reports moved |
| Legacy docs_build report aliases removed | PASS | Removed tracked codex_changed_files.txt and codex_review.diff from docs_build/dev/reports/ |
| References updated within Charlie work only | PASS | Updated moved Charlie Sprite report references only |
| No product planning, architecture, DDL, DML, seed, or intended docs_build docs moved | PASS | Scope limited to generated Charlie report artifacts |
| No runtime behavior changed | PASS | Changed-file scope is reports/artifacts only |
| No Sprite Editor functionality changed | PASS | No assets/toolbox or runtime source changes |
| No DB/API/schema files changed | PASS | Changed-file scope check passed |
| No start_of_day files changed | PASS | Changed-file scope check passed |

## Validation Lane

- PASS: `rg -n "docs_build/dev/reports|docs_build\\dev\\reports|dev/workspace/zip/" dev/reports -g "PR_26179_CHARLIE_0*.md" -g "!PR_26179_CHARLIE_040-dev-artifact-location-cleanup.md"` returned no matches.
- PASS: `git diff --name-only | rg -v "^(dev/reports/|docs_build/dev/reports/)"` returned no out-of-scope paths.
- PASS: `git diff --check` returned no whitespace errors. Git reported CRLF normalization warnings only for moved markdown reports.

## EOD Repair Note

- Rebased against current `origin/main` after PRs 022 through 039 were merged.
- Resolved generated report conflicts by regenerating `dev/reports/codex_changed_files.txt` and `dev/reports/codex_review.diff`.
- Preserved canonical paths: reports under `dev/reports/` and ZIPs under `dev/workspace/zips/`.
- Preserved obsolete tracked `docs_build/dev/reports/` Charlie report removals.

## Manual Validation Notes

- Confirmed this cleanup is artifact-only.
- Confirmed canonical report files now live under dev/reports/.
- Confirmed PR 040 ZIP will be generated under dev/workspace/zips/.
- Confirmed unrelated docs_build/dev/reports/sprite-history-inventory.md remains untouched.

## ZIP

- dev/workspace/zips/PR_26179_CHARLIE_040-dev-artifact-location-cleanup_delta.zip
42 changes: 23 additions & 19 deletions dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,19 +1,23 @@
M dev/build/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md
M dev/build/ProjectInstructions/addendums/pr_workflow.md
M dev/build/ProjectInstructions/repository/canonical_repository_structure.md
A dev/reports/PR277-post-merge-closeout_branch-validation.md
A dev/reports/PR277-post-merge-closeout_manual-validation-notes.md
A dev/reports/PR277-post-merge-closeout_report.md
A dev/reports/PR277-post-merge-closeout_requirement-checklist.md
A dev/reports/PR277-post-merge-closeout_validation-report.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_branch-validation.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_eod-report.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_manual-validation-notes.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_report.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_requirement-checklist.md
A dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_validation-report.md
M dev/reports/codex_changed_files.txt
M dev/reports/codex_review.diff
M dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_branch-validation.md
M dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_eod-report.md
M dev/reports/PR_26179_OWNER_036-reports-are-tracked-source_validation-report.md
dev/reports/PR_26179_CHARLIE_022-sprites-tool-shell.md
dev/reports/PR_26179_CHARLIE_023-sprites-toolbar-placeholders.md
dev/reports/PR_26179_CHARLIE_024-sprites-canvas-grid.md
dev/reports/PR_26179_CHARLIE_025-sprites-basic-drawing.md
dev/reports/PR_26179_CHARLIE_026-sprites-palette-panel.md
dev/reports/PR_26179_CHARLIE_027-sprites-preview-export.md
dev/reports/PR_26179_CHARLIE_028-sprites-editor-polish.md
dev/reports/PR_26179_CHARLIE_029-sprites-clear-reset-controls.md
dev/reports/PR_26179_CHARLIE_030-sprites-undo-redo.md
dev/reports/PR_26179_CHARLIE_031-sprites-picker-zoom.md
dev/reports/PR_26179_CHARLIE_032-sprites-shape-tools.md
dev/reports/PR_26179_CHARLIE_033-sprites-canvas-preview-sync.md
dev/reports/PR_26179_CHARLIE_034-sprites-frame-strip.md
dev/reports/PR_26179_CHARLIE_035-sprites-frame-editing.md
dev/reports/PR_26179_CHARLIE_036-sprites-animation-preview.md
dev/reports/PR_26179_CHARLIE_037-sprites-animation-export.md
dev/reports/PR_26179_CHARLIE_038-sprites-grid-dimension-fix.md
dev/reports/PR_26179_CHARLIE_039-sprites-color-and-zoom-fix.md
dev/reports/PR_26179_CHARLIE_040-dev-artifact-location-cleanup.md
dev/reports/codex_changed_files.txt
dev/reports/codex_review.diff
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading