Skip to content

Fix: enforce recursive object and Slice typing - #129

Merged
0xLeif merged 15 commits into
mainfrom
leif/issue-114-dynamic-object-slice-typing
Jul 29, 2026
Merged

Fix: enforce recursive object and Slice typing#129
0xLeif merged 15 commits into
mainfrom
leif/issue-114-dynamic-object-slice-typing

Conversation

@0xLeif

@0xLeif 0xLeif commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • preserve recursive structural JSON and document JSON-standard integral-number canonicalization
  • validate typed initials, open object shapes, and nonfinite recursive descendants before mutation or output
  • reject present undecodable StoredState objects as corrupt_state while preserving absent-value initials
  • require every Slice to reference an explicitly declared, type-compatible FileState object field
  • advertise every recursive JSON kind in schema contract version 6
  • add repeatable key add --field NAME=TYPE and userSchema.keyCount
  • update README, dynamic-schema design, canonical specs, and Unix/PowerShell smoke coverage

Test Plan

  • fledge lanes run verify
  • 252 serial Swift tests
  • 252 parallel Swift tests
  • expanded Unix smoke
  • PowerShell parser validation and targeted cross-platform contract sequence
  • CHG-0051 and CHG-0056 accepted with exact evidence
  • fledge spec check --strict: 2 specs, 0 errors, 0 warnings
  • hosted CI, Linux smoke, Windows smoke, and Trust on final SHA

Closes #114

Base automatically changed from leif/issue-118-secret-hardening to main July 28, 2026 18:48
@0xLeif
0xLeif force-pushed the leif/issue-114-dynamic-object-slice-typing branch from 55d8d77 to 65b8b5c Compare July 28, 2026 19:29
@0xLeif
0xLeif marked this pull request as ready for review July 28, 2026 19:29
@cursor

cursor Bot commented Jul 28, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 65b8b5c018

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift Outdated
Comment thread Sources/aps/DynamicKeyStorage.swift
Comment thread Sources/aps/Schema.swift
Comment thread Sources/aps/UserSchema.swift

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5ccc63c84

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/DynamicKeyStorage.swift
Comment thread Sources/aps/UserSchema.swift Outdated
Comment thread Sources/aps/UserSchema.swift
Comment thread Sources/aps/UserSchema.swift
Comment thread Sources/aps/UserSchema.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 058730a6bb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/DynamicKeyStorage.swift Outdated
Comment thread Sources/aps/DynamicKeyStorage.swift

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 24aeb9192d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 495b377cfb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/DynamicKeyStorage.swift

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 01128bd888

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fd2b2ef5d5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 28a9781e05

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 256bd9a360

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/aps/UserSchema.swift
@0xLeif
0xLeif merged commit 756fb69 into main Jul 29, 2026
5 checks passed
@0xLeif
0xLeif deleted the leif/issue-114-dynamic-object-slice-typing branch July 29, 2026 00:08
0xLeif added a commit that referenced this pull request Jul 29, 2026
Archive CHG-0051 and CHG-0056 through CHG-0061 after merged PR #129.
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.

Enforce dynamic object and Slice typing and restore schema metadata

1 participant