Correct Asset Tool naming storage dynamic inputs and import layout - PR_26156_154-158-asset-import-corrections & Clean active project upload files when resetting Asset Library - PR_26156_159-asset-local-upload-reset-cleanup

Author: ToolboxAid

.gitignore

@@ -19,6 +19,7 @@ tests/results/
 tmp/test-results/
 tmp/
 scripts/untracked/
+projects/
 
 # Codex files
 docs/dev/codex_commands.md

assets/theme-v2/css/tables.css

@@ -37,35 +37,33 @@ td {
 }
 
 .tool-form-table {
+  table-layout: fixed;
   width: 100%
 }
 
 .tool-form-table th[scope="row"] {
   padding: var(--space-4) var(--space-6);
-  width: 1%;
-  white-space: nowrap;
+  width: 4rem;
+  white-space: normal;
   text-align: right;
   vertical-align: top
 }
 
 .tool-form-table td {
   padding: var(--space-4) var(--space-6);
-  min-width: 12rem;
+  min-width: 0;
   width: auto
 }
 
 .tool-form-table td[colspan] {
-  min-width: 0
+  min-width: 0;
+  overflow-wrap: anywhere
 }
 
 .tool-form-table input:not([type="checkbox"]):not([type="radio"]):not([type="range"]):not([type="color"]),
 .tool-form-table textarea,
 .tool-form-table select {
   box-sizing: border-box;
+  max-width: 100%;
   width: 100%
 }
-
-.data-table tr.is-selected td {
-  background: var(--orange-soft);
-  box-shadow: inset var(--space-3) 0 0 var(--orange)
-}

docs_build/dev/codex_commands.md

@@ -1101,3 +1101,104 @@ Required reports:
 - `docs_build/dev/reports/testing_lane_execution_report.md`
 - `docs_build/dev/reports/codex_changed_files.txt`
 - `docs_build/dev/reports/codex_review.diff`
+
+
+## PR_26156_154-158
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Read `archive/v1-v2/tools/old_asset-manager-v2/js/assetManagerMetadata.js` as functionality reference only.
+- Updated Asset Tool section order so Library Records is primary and Asset Roles is supporting information below it.
+- Kept singular `Asset Role` for the form field and plural `Asset Roles` for the all-roles list.
+- Updated Storage Path generation to `assets/projects/<projectId>/<assetRole>/<usage>/<filename>`.
+- Preserved project-owned upload storage only.
+- Restored file-based import behavior for Audio, Data, Font, Image, Localization, Shader, and Video.
+- Added Color palette-prep behavior with visible `Palette Tool required.` diagnostics.
+- Fixed Import Asset table layout through reusable Theme V2 table styling.
+- Rolled back selected row highlighting and highlighted only the selected record button.
+- Did not copy archive/reference code.
+- Did not modify archived V1/V2 files.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/assets/assets-mock-repository.js`
+- `node --check toolbox/assets/assets.js`
+- `node --check tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+- `node ./scripts/run-targeted-test-lanes.mjs --lane asset-tool`
+- `rg --pcre2 -n "<style|style=|on(click|change|input|submit)=|<script(?![^>]*\\bsrc=)" toolbox/assets/index.html toolbox/assets/assets.js toolbox/assets/assets-mock-repository.js assets/theme-v2/css/tables.css tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+- `git diff --check`
+- Full samples smoke: skipped by request and because samples were not changed.
+
+Required reports:
+- `docs_build/dev/reports/asset-import-correction-stacked-report.md`
+- `docs_build/dev/reports/testing_lane_execution_report.md`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`
+
+
+## PR_26156_159
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Updated Asset Tool simulated upload paths to `projects/<projectId>/<assetRole>/<usage>/<filename>`.
+- Resolved the Asset Tool demo project storage id to the ULID `01K8M3K0EX7V5A3W9Q2Y6R4T1B`.
+- Added `projects/` to `.gitignore`.
+- Updated Reset Asset Library so it clears active project mock file/storage rows and associated Asset Tool metadata.
+- Updated Reset Asset Library to fail visibly when no active project id exists.
+- Updated Reset Asset Library to report deleted simulated file/folder counts.
+- Preserved the browser-only/mock storage boundary; no real disk deletion was added.
+- Did not modify archived V1/V2 files.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/assets/assets-mock-repository.js`
+- `node --check toolbox/assets/assets.js`
+- `node --check tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+- `node ./scripts/run-targeted-test-lanes.mjs --lane asset-tool`
+- `rg --pcre2 -n "<style|style=|on(click|change|input|submit)=|<script(?![^>]*\\bsrc=)" toolbox/assets/index.html toolbox/assets/assets.js toolbox/assets/assets-mock-repository.js assets/theme-v2/css/tables.css tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+- `rg -n "assets/projects" toolbox/assets tests/playwright/tools/AssetToolMockRepository.spec.mjs docs_build/dev/reports/asset-import-correction-stacked-report.md docs_build/dev/reports/asset-local-upload-reset-cleanup-report.md`
+- `git diff --check`
+- Full samples smoke: skipped by request and because samples were not changed.
+
+Required reports:
+- `docs_build/dev/reports/asset-local-upload-reset-cleanup-report.md`
+- `docs_build/dev/reports/testing_lane_execution_report.md`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`
+
+
+## PR_26156_160-163
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Read archived Asset Manager V2 reference files only for functionality expectations:
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/controls/AssetFormControl.js`
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/assetManagerMetadata.js`
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/services/WorkspaceBridge.js`
+- Added explicit Asset Tool picker modes: `file`, `palette`, `managed-tool`, and `advanced`.
+- Added visible Picker Mode and Palette Color rows to the Import Asset form.
+- Updated role switching so Usage options, file `accept`, picker visibility, storage path preview, and diagnostics update together.
+- Changed Color to palette mode with visible `Palette Tool required.` diagnostic.
+- Changed Data and Localization to managed-tool mode with visible tool-required diagnostics.
+- Changed Shader to advanced mode and hid it from normal users unless Admin/Advanced mode is active.
+- Kept Storage Path generated/read-only as `projects/<projectId>/<assetRole>/<usage>/<filename>`.
+- Updated targeted Asset Tool Playwright coverage for dynamic picker switching and advanced/admin-only Shader visibility.
+- Added no CSS.
+- Did not copy archived/reference code.
+- Did not modify archived V1/V2 files.
+- Did not modify `start_of_day`.
+
+Validation:
+- `node --check toolbox/assets/assets-mock-repository.js`
+- `node --check toolbox/assets/assets.js`
+- `node --check tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+- `node ./scripts/run-targeted-test-lanes.mjs --lane asset-tool`
+- `Select-String -Path toolbox/assets/index.html,toolbox/assets/assets.js -Pattern '<style|style=|<script(?![^>]*\\bsrc=)|on(click|change|input|submit)=' -CaseSensitive:$false`
+- `git diff --check`
+- Full samples smoke: skipped by request and because samples were not changed.
+
+Required reports:
+- `docs_build/dev/reports/asset-dynamic-picker-model-report.md`
+- `docs_build/dev/reports/testing_lane_execution_report.md`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`

docs_build/dev/commit_comment.txt

@@ -1 +1 @@
-Polish Asset library selection, import layout, and metadata - PR_26156_151-153
+Update Asset dynamic picker model - PR_26156_160-163-asset-dynamic-picker-model

docs_build/dev/reports/asset-dynamic-picker-model-report.md

@@ -0,0 +1,104 @@
+# PR_26156_160-163 Asset Dynamic Picker Model Report
+
+## Scope
+- Built stacked PRs in order:
+  - `PR_26156_160-asset-dynamic-picker-model`
+  - `PR_26156_161-asset-color-palette-picker`
+  - `PR_26156_162-asset-file-picker-role-behavior`
+  - `PR_26156_163-asset-dynamic-picker-validation`
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` first.
+- Used archived Asset Manager V2 files as behavior reference only:
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/controls/AssetFormControl.js`
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/assetManagerMetadata.js`
+  - `archive/v1-v2/tools/old_asset-manager-v2/js/services/WorkspaceBridge.js`
+- Did not copy archived/reference code.
+- Did not modify archived V1/V2 files.
+- Did not modify `start_of_day`.
+- Added no CSS.
+
+## PR_26156_160 Dynamic Picker Model
+- Added explicit Asset Tool picker modes:
+  - `file`
+  - `palette`
+  - `managed-tool`
+  - `advanced`
+- Made picker mode visible in the Import Asset form.
+- Updated role changes to refresh:
+  - Usage options.
+  - File `accept` values.
+  - Visible picker mode.
+  - File/palette picker visibility.
+  - Import diagnostics.
+- Removed silent fallback from non-file roles.
+
+## PR_26156_161 Color Palette Picker
+- Color now uses `palette` picker mode.
+- Color hides the file picker.
+- Color shows a visible palette color picker row.
+- Color keeps Usage visible for palette category assignment.
+- Color shows `Palette Tool required.` when no active Project Workspace palette exists.
+- Project Workspace palette handoff remains future work; no Project Workspace implementation was changed in this PR.
+
+## PR_26156_162 File Picker Role Behavior
+- File picker remains visible only for:
+  - Image
+  - Audio
+  - Video
+  - Font
+- File `accept` values now update by selected file role:
+  - Image: `.png,.jpg,.jpeg,.webp,.gif,.svg`
+  - Audio: `.mp3,.wav,.ogg,.m4a`
+  - Video: `.mp4,.webm,.mov`
+  - Font: `.ttf,.otf,.woff,.woff2`
+- Data now uses `managed-tool` mode with `Data/Table Tool required.`
+- Localization now uses `managed-tool` mode with `Localization Tool required.`
+- Shader now uses `advanced` mode.
+- Shader is hidden from normal users and appears only with Admin/Advanced mode.
+- Storage Path remains generated/read-only as:
+  - `projects/<projectId>/<assetRole>/<usage>/<filename>`
+- The active demo project id remains the ULID:
+  - `01K8M3K0EX7V5A3W9Q2Y6R4T1B`
+
+## PR_26156_163 Validation
+- Updated targeted Asset Tool Playwright coverage for:
+  - Dynamic picker switching.
+  - Color hiding file picker and showing Palette Tool requirement.
+  - Image/Audio/Video/Font file picker visibility and `accept` values.
+  - Data/Localization managed-tool diagnostics.
+  - Shader admin/advanced-only visibility.
+  - No normal-user import/export JSON workflow.
+- Preserved existing upload success coverage for Image, Video, and Audio.
+- Preserved project-owned path and reset coverage from prior PRs.
+
+## Validation Notes
+- Impacted lane: `asset-tool`.
+- Ran:
+  - `node --check toolbox/assets/assets-mock-repository.js`
+  - `node --check toolbox/assets/assets.js`
+  - `node --check tests/playwright/tools/AssetToolMockRepository.spec.mjs`
+  - `node ./scripts/run-targeted-test-lanes.mjs --lane asset-tool`
+  - `Select-String -Path toolbox/assets/index.html,toolbox/assets/assets.js -Pattern '<style|style=|<script(?![^>]*\\bsrc=)|on(click|change|input|submit)=' -CaseSensitive:$false`
+- Result:
+  - Targeted Asset Tool lane passed: 5/5.
+  - Changed-file syntax checks passed.
+  - Inline style/script/event-handler scan passed.
+
+## Skipped Lanes
+- Project Workspace palette handoff lane skipped because this PR only adds the missing palette diagnostic and does not change Project Workspace behavior.
+- Full samples smoke skipped by request and because samples were not changed.
+- Broad workspace/navigation lanes skipped because no shared launch, navigation, Theme V2 CSS, parser, DB, or cross-tool runtime behavior changed.
+
+## Manual Test Notes
+- Verified through the targeted Playwright lane:
+  - Asset page loads without console/page/request failures.
+  - Normal role view hides Shader.
+  - Admin role view exposes Shader in `advanced` picker mode.
+  - Color uses `palette` picker mode and shows Palette Tool required.
+  - Data and Localization use `managed-tool` diagnostics.
+  - File-based roles show file picker and role-specific `accept` values.
+  - Normal user-facing output does not add import/export JSON workflow.
+
+## Theme V2 Gap Findings
+- No Theme V2 gap found.
+- Existing Theme V2 form table/layout classes supported the added picker rows.
+- No new CSS was added.