feat: add @script-development/fs-adapter-store package#8
Merged
Conversation
Reactive adapter-store pattern with domain state management, CRUD resource adapters, caching, and localStorage persistence. Uses reactive getter pattern for existing resources. Generic New<T> defaults to Omit<T, 'id'> — territories override. No case conversion — middleware handles snake/camel transformation. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This was referenced Apr 30, 2026
Merged
jasperboerhof
added a commit
that referenced
this pull request
May 1, 2026
…t-surface feat(fs-http): timeout option with 30000ms default (Doctrine #8 library-author extension)
This was referenced May 4, 2026
Goosterhof
added a commit
that referenced
this pull request
May 7, 2026
…-http-docs docs(fs-http): surface Doctrine #8 timeout contract; collapse README duplication
3 tasks
Goosterhof
added a commit
that referenced
this pull request
May 7, 2026
…ctrine Four edits to CLAUDE.md, sourced from Adjutant M2 doctrine inquiry (2026-05-07): - Add `## Versioning Discipline (Pre-1.0)` — encodes the mechanical per-bump checklist (widen ranges, patch-bump siblings, regenerate lock, verify no nested registry copies). Notes that `workspace:*` is not an option on npm (rejected as EUNSUPPORTEDPROTOCOL). - Add `### Internal Dependency Coordination` sub-section under `## Conventions` — captures the string-ts version-sync discipline between fs-helpers and fs-translation. Reframes the "duplicate bundle" claim against verified npm dedupe behavior. - Add `## Doctrine #8 — HTTP Timeout Surface (fs-http)` — surfaces fs-http as the war-room reference implementation of the library-author extension to Architectural Principle #8 (verbatim quote from war-room CLAUDE.md). Documents the 5-axis surface. - Fix TypeScript version line: `5.9+` → `^6.0` (matches root package.json#devDependencies.typescript). No code or test changes; markdown only. Order: orders/fs-packages/claude-md-doctrine-consolidation-engineer-deployment.md Adjutant field report: reports/fs-packages/field/2026-05-07-adjutant-doctrine-inquiry-cascade-tax-string-ts.md Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Goosterhof
added a commit
that referenced
this pull request
May 8, 2026
Add an Authentication & XSRF section to docs/packages/http.md sibling- level to the Doctrine #8 Timeout block, naming Laravel Sanctum SPA explicitly, explaining the XSRF-TOKEN cookie + axios 1.x withXSRFToken mechanics, and stating the HTTP 419 failure mode (which mocked transports do not surface, per ADR-0017). Stateless / token / non- Sanctum stacks documented as the inverse case. Expand the API Reference row for options.withXSRFToken with the same guidance and a cross-link to the new section. Add a one-line README pointer mirroring the Timeout pointer shape landed by Engineer PR #68 (no full duplication; docs page is source of truth). Closes enforcement queue #22 docs path. Commander disposition: docs- only. Library default of withXSRFToken: false stays as-is. No oxlint call-site rule. No source code changes. No version bump. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced May 15, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
() => T) for existing resources — mutations flow through adapter methods, not direct store accessNew<T>defaults toOmit<T, "id">— territories override with their own omissionsTest plan
🤖 Generated with Claude Code