Repository navigation
Spiral/cyc 20260926 ttz7 c 63 schema identity - #28
Merged
Merged
Conversation
Retain the maintainer's report and stable-identity direction for SRC-20260926-TTZ7C-61. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Record DEF-20260926-TTZ7C-62 (schema objects identified by path across the isolate) and open CYC-20260926-TTZ7C-63 as a deliberate roadmap deviation to correct it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Baseline for DEF-20260926-TTZ7C-62: shared, linked and cyclic schema objects must keep identity across the query isolate, and JSONTag responses must match host serialization. Both tests fail on bd61d9c (identity false; deeply shared schema times out). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Correct DES-20260924-TTZ7C-59 for DEF-20260926-TTZ7C-62: schema paths do not identify shared or cyclic schema objects. Number schema objects once by a deterministic walk, keep the handles private to the bridge, and keep record numbers as data identity. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Retain the maintainer's choice to preserve schema JSONTag ids for query workers, and bring guest-side read caching into CYC-20260926-TTZ7C-63 after measuring 2.7 s for the full schema as JSON. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Extend the DES-20260924-TTZ7C-59 correction: query workers parse the schema's JSONTag text so ids survive, and guest proxies cache reads for the query's lifetime. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Implement the corrected DES-20260924-TTZ7C-59 for DEF-20260926-TTZ7C-62: - number schema objects once and reference them by private handles; - pass the schema's JSONTag text to query workers so ids survive; - cache host reads per guest view for the query's lifetime. curriculum-store's schema now returns in 34 ms as JSONTag (matching host serialization) and 271 ms as JSON, where it previously exhausted the isolate or timed out. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Evidence for the maintainer's requirement in SRC-20260926-TTZ7C-61 that schema identity outlives a single query. Rewrap the query runtime note. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
EVD-20260926-TTZ7C-64: 205 passing tests, failing baselines, and curriculum-store's schema at 34 ms (JSONTag) / 271 ms (JSON). Record the DEF-20260926-TTZ7C-62 correction and retained discoveries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Causal-chain review before acceptance: - DES-20260924-TTZ7C-59 is constrained by DEF-20260926-TTZ7C-62; sd:changeCausedBy is reserved for implementations. - IMP-20260919-TTZ7C-33's transition also cites SRC-20260926-TTZ7C-61. - EVD-20260926-TTZ7C-64's intermediate timings included schema ids. - Project context records CYC-56 as integrated and CYC-63 as active. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Retain the maintainer's request for object identity across all query data and the decision to derive output ids from record numbers. Shared arrays and the jsontag fallback bug are deferred. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
DES-20260924-TTZ7C-59: serialize shared objects without ids using record-derived or per-response output ids, supplied only during serialization, and copy tagged scalars once per query. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Shared stored and query-built objects without ids must link in JSONTag responses with record-derived or per-response ids, hidden from queries; tagged scalars must keep identity within a query. Both fail on 832afc1 (crypto is not defined; scalar identity false). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Implement DES-20260924-TTZ7C-59 for SRC-20260926-TTZ7C-61: - before JSONTag serialization, assign output ids to shared objects without an id: ~<record> for views, supplied only while serializing, and ~query-<n> for query-built objects; avoid stored ids; - copy each tagged scalar once per view property and query. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
EVD-20260926-TTZ7C-64: regressions, 207 passing tests, stable output ids across updates, and about 4% added cost on large scans. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Record the maintainer's acceptance of CYC-20260926-TTZ7C-63 at fa6c791 and update the project context. Integration remains a separate decision. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
No description provided.