PR_26177_CHARLIE_012-sprites-library-crud

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-branch-validation.md

@@ -0,0 +1,18 @@
+# PR_26177_CHARLIE_012 Branch Validation
+
+Status: PASS
+
+## Checks
+
+- PASS: `main` was checked and synced to `origin/main` before continuing.
+- PASS: `main` worktree was clean before continuing.
+- PASS: PR012 was created as a stacked branch from `PR_26177_CHARLIE_011-sprites-tool-shell`.
+- PASS: Stacking is required because CRUD depends on the PR011 shell and avoids duplicating PR011 in a separate main-based branch.
+- PASS: Current work branch is `PR_26177_CHARLIE_012-sprites-library-crud`.
+- PASS: Branch contains only the Sprites library CRUD PR scope relative to PR011.
+- PASS: No merge was performed.
+- PASS: No `start_of_day` path is changed.
+
+## Notes
+
+The Sprites API/database foundation is still provided by `PR_26177_CHARLIE_010-sprites-api-db-foundation`. This PR validates browser CRUD behavior against mocked API responses matching that contract.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-manual-validation-notes.md

@@ -0,0 +1,20 @@
+# PR_26177_CHARLIE_012 Manual Validation Notes
+
+Status: PASS
+
+## Manual Review
+
+- Verified Add Sprite opens an editable row without generating a sprite key in the browser.
+- Verified Save requires a sprite name.
+- Verified Save requires an explicit valid status.
+- Verified category input is normalized before API submission.
+- Verified create and update requests use the Sprites API POST contract.
+- Verified archive uses the API archive endpoint.
+- Verified delete uses the API delete endpoint only when the record has no usage references.
+- Verified referenced records display a disabled delete action with archive as the available safe action.
+- Verified 401 write response redirects to `account/sign-in.html`.
+- Verified no color definitions or reusable Palette/Colors records are created by Sprites.
+
+## Manual Limitation
+
+The live API/database foundation remains in PR010. This PR uses mocked API responses in Playwright to validate the UI contract until PR010 is merged.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-requirements-checklist.md

@@ -0,0 +1,25 @@
+# PR_26177_CHARLIE_012 Requirements Checklist
+
+Status: PASS
+
+- PASS: Implemented API-backed sprite library table controls.
+- PASS: Added create flow.
+- PASS: Added edit/update flow.
+- PASS: Added archive flow.
+- PASS: Added delete flow.
+- PASS: Delete is blocked for referenced records using API-provided usage count.
+- PASS: Archive remains available as the safer referenced-record action.
+- PASS: Name is required before save.
+- PASS: Status must be one of the API contract values: `draft`, `ready`, `published`, `archived`.
+- PASS: Category is optional and normalized.
+- PASS: No silent default status is applied for new records.
+- PASS: Guest save attempts redirect to `account/sign-in.html`.
+- PASS: Browser does not generate authoritative sprite keys.
+- PASS: API/server remains responsible for key and audit fields.
+- PASS: No Sprite-owned reusable color data was added.
+- PASS: No page-local reusable Palette/Colors arrays were added.
+- PASS: No browser storage product-data source of truth was added.
+- PASS: No MEM DB, local-mem, fake-login, or silent fallback was introduced.
+- PASS: Targeted Playwright coverage passed.
+- PASS: Required report artifacts were created.
+- PASS: Repo-structured ZIP artifact was created under `tmp/`.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-validation-lane.md

@@ -0,0 +1,40 @@
+# PR_26177_CHARLIE_012 Validation Lane
+
+Status: PASS
+
+## Commands
+
+```powershell
+git -c http.sslBackend=schannel pull --ff-only origin main
+git rev-list --left-right --count origin/main...HEAD
+```
+
+Result: PASS, `main` was synced at `0 0` before the stacked branch was created.
+
+```powershell
+rg -n "<style|style=|onclick=|onchange=|oninput=|onsubmit=|<script>" toolbox/sprites/index.html assets/toolbox/sprites/js/index.js tests/playwright/tools/SpritesToolShell.spec.mjs
+```
+
+Result: PASS, no matches.
+
+```powershell
+rg -n "localStorage|sessionStorage|indexedDB|imageDataUrl|MEM DB|local-mem|fake-login|silent fallback" toolbox/sprites/index.html assets/toolbox/sprites/js/index.js tests/playwright/tools/SpritesToolShell.spec.mjs
+```
+
+Result: PASS, no matches.
+
+```powershell
+git diff --check
+```
+
+Result: PASS. Git reported only repository line-ending warnings for changed HTML/test files.
+
+```powershell
+node ./node_modules/@playwright/test/cli.js test tests/playwright/tools/SpritesToolShell.spec.mjs --project=playwright --workers=1 --reporter=list
+```
+
+Result: PASS, 6 passed.
+
+## Playwright Coverage
+
+Targeted Playwright coverage updated `docs_build/dev/reports/playwright_v8_coverage_report.txt` for the Sprites browser module.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud.md

@@ -0,0 +1,51 @@
+# PR_26177_CHARLIE_012-sprites-library-crud
+
+Team: Charlie
+
+Status: PASS
+
+## Scope
+
+Extended the Sprites tool shell with API-backed library CRUD controls. The browser still does not own authoritative data: all create, update, archive, and delete actions call the Sprites API contract.
+
+## Changed Files
+
+- `toolbox/sprites/index.html`
+- `assets/toolbox/sprites/js/index.js`
+- `tests/playwright/tools/SpritesToolShell.spec.mjs`
+- `docs_build/dev/reports/playwright_v8_coverage_report.txt`
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-branch-validation.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-requirements-checklist.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-validation-lane.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-manual-validation-notes.md`
+
+## Implementation Notes
+
+- Added Add/Edit/Save/Cancel controls for sprite records.
+- Added required `name` validation and API-contract status validation.
+- Added category normalization by trimming and collapsing internal whitespace.
+- Added archive action through `/api/sprites/records/:key/archive`.
+- Added delete action through `/api/sprites/records/:key/delete`.
+- Delete is disabled when the API-provided usage count indicates references; archive remains the safer action.
+- Guest write attempts redirect to `account/sign-in.html` when the API returns 401 or 403.
+- Create requests do not include browser-generated authoritative keys.
+- Sprites still does not define reusable colors or duplicate Palette/Colors records.
+
+## Stack Note
+
+This PR is stacked on `PR_26177_CHARLIE_011-sprites-tool-shell` because the CRUD work depends on the shell structure. Project Instructions allow stacked sequential PRs when dependency order requires it. Opening this from `main` would duplicate PR011 shell work and violate one-purpose scope.
+
+## Validation
+
+- PASS: `git diff --check`
+- PASS: inline CSS/script/handler scan for Sprites files found no matches.
+- PASS: browser storage and forbidden local data pattern scan found no matches.
+- PASS: no `start_of_day` files changed.
+- PASS: `node ./node_modules/@playwright/test/cli.js test tests/playwright/tools/SpritesToolShell.spec.mjs --project=playwright --workers=1 --reporter=list`
+
+## ZIP Artifact
+
+- `tmp/PR_26177_CHARLIE_012-sprites-library-crud_delta.zip`

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud_branch-validation.md

@@ -0,0 +1,23 @@
+# PR_26177_CHARLIE_012-sprites-library-crud
+
+Generated: 2026-06-26
+Team: Charlie
+GitHub PR: #222
+Branch: PR_26177_CHARLIE_012-sprites-library-crud
+Base: PR_26177_CHARLIE_011-sprites-tool-shell
+
+## Branch Validation
+
+| Check | Result | Notes |
+| --- | --- | --- |
+| Branch exists locally | PASS | PR_26177_CHARLIE_012-sprites-library-crud |
+| Branch pushed to origin | PASS | Existing upstream branch is present. |
+| Worktree clean before report generation | PASS | Verified before companion reports were written. |
+| Runtime/UI/API/database change in this report commit | PASS | None; report-only completion. |
+| start_of_day changes | PASS | None. |
+| GitHub draft state | PENDING | PR remains draft until Owner review. |
+| GitHub mergeable state at EOD snapshot | PASS | mergeable=true; not merged because no Owner approval was provided. |
+
+## Result
+
+Report artifact completion is PASS. Merge readiness remains pending Owner review and dependency-order workflow.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud_manual-validation-notes.md

@@ -0,0 +1,19 @@
+# PR_26177_CHARLIE_012-sprites-library-crud
+
+Generated: 2026-06-26
+Team: Charlie
+GitHub PR: #222
+Branch: PR_26177_CHARLIE_012-sprites-library-crud
+Base: PR_26177_CHARLIE_011-sprites-tool-shell
+
+## Manual Validation Notes
+
+- PR scope: Adds API-backed Sprites create, edit, archive, and guarded delete controls.
+- Manual review should verify the PR in dependency order after prior Sprites branches are approved or merged.
+- Browser-owned product data, page-local reusable color arrays, and silent fallback behavior should remain absent.
+- Palette/Colors must remain the authoritative owner of reusable colors; Sprites may reference color keys only.
+- Because this EOD update is report-only, no new manual UI behavior was introduced.
+
+## EOD Status
+
+Open draft PR. Not merged. Awaiting Owner review and dependency-order workflow.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud_requirements-checklist.md

@@ -0,0 +1,24 @@
+# PR_26177_CHARLIE_012-sprites-library-crud
+
+Generated: 2026-06-26
+Team: Charlie
+GitHub PR: #222
+Branch: PR_26177_CHARLIE_012-sprites-library-crud
+Base: PR_26177_CHARLIE_011-sprites-tool-shell
+
+## Requirement Checklist
+
+| Requirement | Result | Notes |
+| --- | --- | --- |
+| One PR purpose only | PASS | Adds API-backed Sprites create, edit, archive, and guarded delete controls. |
+| Theme V2 / current GFS patterns preserved | PASS | Validated in original PR lane where UI was touched. |
+| Browser -> API -> Database flow preserved | PASS | No browser-owned authoritative product data added. |
+| Palette/Colors remains color SSoT | PASS | Sprites references Palette/Colors keys only where applicable. |
+| No MEM DB/local-mem/fake-login/silent fallback introduced | PASS | No forbidden runtime patterns introduced in original PR scope. |
+| No start_of_day changes | PASS | Confirmed for report completion. |
+| Required companion reports present | PASS | Branch validation, checklist, validation lane, and manual notes added by this EOD report-only commit. |
+| Repo ZIP under tmp | PASS | Local EOD ZIP was regenerated under tmp for this PR. |
+
+## Result
+
+PASS for report completion. PR remains unmerged pending review/approval.

docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud_validation-lane.md

@@ -0,0 +1,26 @@
+# PR_26177_CHARLIE_012-sprites-library-crud
+
+Generated: 2026-06-26
+Team: Charlie
+GitHub PR: #222
+Branch: PR_26177_CHARLIE_012-sprites-library-crud
+Base: PR_26177_CHARLIE_011-sprites-tool-shell
+
+## Validation Lane
+
+Original targeted validation recorded by the PR:
+
+- git diff --check
+- inline style/script/handler scan
+- browser storage/forbidden local data scan
+- Playwright SpritesToolShell (6 passed)
+
+EOD report-completion validation:
+
+- PASS: report-only changed-file intent.
+- PASS: no source/runtime files changed by this companion report commit.
+- PASS: ZIP artifact regenerated locally under tmp.
+
+## Result
+
+PASS for documented targeted validation. No additional runtime validation was run during EOD report generation because no implementation files changed.

docs_build/dev/reports/codex_changed_files.txt

@@ -1,28 +1,24 @@
 # git status --short
-M docs_build/dev/reports/codex_changed_files.txt
- M docs_build/dev/reports/codex_review.diff
+M assets/toolbox/sprites/js/index.js
  M docs_build/dev/reports/playwright_v8_coverage_report.txt
+ M tests/playwright/tools/SpritesToolShell.spec.mjs
  M toolbox/sprites/index.html
-?? assets/toolbox/sprites/
-?? docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-branch-validation.md
-?? docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-manual-validation-notes.md
-?? docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-requirements-checklist.md
-?? docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-validation-lane.md
-?? docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell.md
-?? tests/playwright/tools/SpritesToolShell.spec.mjs
+?? docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-branch-validation.md
+?? docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-manual-validation-notes.md
+?? docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-requirements-checklist.md
+?? docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-validation-lane.md
+?? docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud.md
 
 # git ls-files --others --exclude-standard
-assets/toolbox/sprites/js/index.js
-docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-branch-validation.md
-docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-manual-validation-notes.md
-docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-requirements-checklist.md
-docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell-validation-lane.md
-docs_build/dev/reports/PR_26177_CHARLIE_011-sprites-tool-shell.md
-tests/playwright/tools/SpritesToolShell.spec.mjs
+docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-branch-validation.md
+docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-manual-validation-notes.md
+docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-requirements-checklist.md
+docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud-validation-lane.md
+docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud.md
 
 # git diff --stat
-docs_build/dev/reports/codex_changed_files.txt     |   40 +-
- docs_build/dev/reports/codex_review.diff           | 2052 +++++++-------------
- .../dev/reports/playwright_v8_coverage_report.txt  |   15 +-
- toolbox/sprites/index.html                         |   74 +-
- 4 files changed, 770 insertions(+), 1411 deletions(-)
+assets/toolbox/sprites/js/index.js                 | 314 ++++++++++++++++++++-
+ .../dev/reports/playwright_v8_coverage_report.txt  |  10 +-
+ tests/playwright/tools/SpritesToolShell.spec.mjs   | 148 ++++++++++
+ toolbox/sprites/index.html                         |   6 +-
+ 4 files changed, 465 insertions(+), 13 deletions(-)