feat(web): ADR-083 Phase 8 - Web Client Artifact Integration #198
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e 8) - Create useArtifactStore with localStorage LRU caching (50MB limit) - Create useQueryDefinitionStore for saved queries - Add artifact API methods to web client (list, get, create, delete, regenerate) - Add "Save as Artifact" button to PolarityExplorerWorkspace - Create ArtifactStatusBadge and StaleIndicator components
…R-083 Phase 8) Rewrites blockDiagramStore and reportStore to use artifacts and query_definitions APIs as primary persistence, with localStorage serving as cache/fallback. Changes: - blockDiagramStore: API-first persistence with query_definitions endpoint - All methods now async (saveDiagram, loadDiagram, deleteDiagram) - Added migrateFromLocalStorage() for one-time migration - localStorage serves as offline fallback - reportStore: API-first persistence with artifacts endpoint - Reports stored as artifacts with type 'report' - Added loadReports() and migrateFromLocalStorage() - Maintains local state for immediate UI updates - Updated all consuming components for async operations: - BlockBuilder: async save/load/delete handlers - BlockEditorWorkspace: async diagram selection and deletion - ReportWorkspace: async delete and rename - ExplorerView: async handleSendToReports - PolarityExplorerWorkspace: async handleSendToReports - Removed completed ADR-081 tracking file
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Completes Phase 8 of ADR-083 (Artifact Persistence Pattern) by integrating the web client with the artifact and query definition APIs.
useArtifactStoreanduseQueryDefinitionStorefor managing artifact metadata and query definitionsblockDiagramStoreandreportStoreto use APIs with migration utilities for existing localStorage dataNew Files
web/src/types/artifacts.ts- TypeScript types for artifact systemweb/src/store/artifactStore.ts- Artifact state management with LRU cachingweb/src/store/queryDefinitionStore.ts- Query definition state managementweb/src/components/shared/ArtifactStatusBadge.tsx- Freshness indicator componentsAPI-First Pattern
Stores now follow an API-first pattern:
Test plan
migrateFromLocalStorage()utility for existing dataRelated