Skip to content

PR_26177_CHARLIE_012-sprites-library-crud#222

Draft
ToolboxAid wants to merge 2 commits into
PR_26177_CHARLIE_011-sprites-tool-shellfrom
PR_26177_CHARLIE_012-sprites-library-crud
Draft

PR_26177_CHARLIE_012-sprites-library-crud#222
ToolboxAid wants to merge 2 commits into
PR_26177_CHARLIE_011-sprites-tool-shellfrom
PR_26177_CHARLIE_012-sprites-library-crud

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

  • Extends the Sprites shell with API-backed create, edit, archive, and delete controls.
  • Validates name and explicit API-contract status before saves, normalizes optional category text, and avoids browser-generated authoritative keys.
  • Redirects guest write attempts to account/sign-in.html and blocks destructive delete for referenced records while preserving archive as the safer action.

Stack

  • Base: PR_26177_CHARLIE_011-sprites-tool-shell
  • Depends on API foundation from PR_26177_CHARLIE_010-sprites-api-db-foundation for live backend behavior.

Validation

  • git diff --check
  • Inline style/script/handler scan for Sprites files
  • Browser storage / forbidden local data pattern scan
  • node ./node_modules/@playwright/test/cli.js test tests/playwright/tools/SpritesToolShell.spec.mjs --project=playwright --workers=1 --reporter=list (6 passed)

Artifacts

  • tmp/PR_26177_CHARLIE_012-sprites-library-crud_delta.zip
  • docs_build/dev/reports/PR_26177_CHARLIE_012-sprites-library-crud.md
  • docs_build/dev/reports/codex_review.diff
  • docs_build/dev/reports/codex_changed_files.txt

Draft until Owner review.

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.

1 participant