Skip to content

Add conditional transformation profile 0.2 - #5

Merged
TonisOrmisson merged 79 commits into
mainfrom
agent/conditional-transformation-0.2
Aug 4, 2026
Merged

Add conditional transformation profile 0.2#5
TonisOrmisson merged 79 commits into
mainfrom
agent/conditional-transformation-0.2

Conversation

@TonisOrmisson

@TonisOrmisson TonisOrmisson commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

What changed

  • defines the optional Transformation Plan 0.2 contract with conditional assignment, typed operands, and ordered execution semantics
  • defines the bounded SPSS syntax frontend 0.2 and its SQL in-place binding
  • adds normative documentation, machine-readable JSON Schemas, and declarative conformance fixtures
  • keeps successful transformations in place: dataset identity and physical wide-table identity are preserved
  • keeps the work under Unreleased, planned for specification v0.3.0

Architecture boundary

This specification repository contains normative documentation, schemas, and declarative fixtures only for Transformation Plan/frontend conformance.

It does not contain a parser, semantic interpreter, SQL executor, adapter reference implementation, or executable Transformation Plan conformance harness. Behavioral validation belongs in implementation and adapter repositories.

Validation

  • existing repository validation passes locally
  • exact-head push CI: run 30896274513
  • exact-head pull-request CI: run 30896282641
  • published 0.1 conformance manifests remain byte-for-byte unchanged

Status

Draft release-candidate specification work. The PR is not claiming executable adapter conformance.

Copy link
Copy Markdown
Contributor Author

@codex review

@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: b56eb9a316

ℹ️ 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 conformance/in-place-transformation-0.2.json
Comment thread docs/spss-syntax-frontend-profile-0.2.md

Copy link
Copy Markdown
Contributor Author

@codex review

@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: bddd110325

ℹ️ 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 docs/transformation-plan-profile-0.2.md Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

@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: 5a8c2ff978

ℹ️ 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 docs/transformation-plan-sql-binding-0.2.md Outdated
Comment thread tools/validate_transformation_plan_0_2.py Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

@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: 4d2e5b2cf1

ℹ️ 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 docs/spss-syntax-frontend-profile-0.2.md
Comment thread conformance/spss-syntax-frontend-0.2.json

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 8d47ce2c152bcc19b4281280c4ca517d26661f21, with particular attention to the specification/implementation boundary: Transformation Plan and SPSS frontend behavioral interpreters were removed; the remaining Python must stay generic structural and integrity validation only.

@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: 8d47ce2c15

ℹ️ 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 tools/validate_repository.py Outdated
Comment thread tools/validate_repository.py Outdated
Comment thread tools/validate_repository.py Outdated

Copy link
Copy Markdown
Contributor Author

Resolved the three exact-head 8d47ce2 review findings in f09cac0882362caeafee1e46042138ea9370848c:

  • all six manifest identity declarations (including Plan 0.2 canonicalization) are checked generically;
  • source hashes normalize CRLF/bare CR to LF in frontend and linked-binding checks;
  • failed frontend fixtures cannot carry partial plan artifacts or a non-null plan hash.

No behavioral interpreter or case-ID allowlist was added. Local validation and exact-head push/PR CI are green.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head f09cac0882362caeafee1e46042138ea9370848c. Preserve the documented architecture boundary: schemas and declarative conformance fixtures live here; behavioral parser/SQL/adapter interpreters belong in implementation repositories. The remaining Python is intended to be generic structural and referential-integrity validation only.

@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: f09cac0882

ℹ️ 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 conformance/transformation-plan-0.1.json Outdated

Copy link
Copy Markdown
Contributor Author

Fixed the exact-head 0.1 immutability finding in 2a5713ab15d72cf18521b965b1698d48d36cc3d7. All three published 0.1 conformance manifests are byte-for-byte identical to their pre-cleanup 6ba206a versions (git diff --exit-code 6ba206a..HEAD -- for those files is empty). The artificial legacy binding cross-links were removed; generic validation remains compatible with the unchanged artifacts. Exact-head push and PR CI are green.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 2a5713ab15d72cf18521b965b1698d48d36cc3d7. The published 0.1 conformance manifests are byte-for-byte restored and immutable; the prior finding is fixed. Preserve the documented boundary: no behavioral interpreter belongs in this specification repository.

@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: 2a5713ab15

ℹ️ 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 conformance/in-place-transformation-0.2.json
Comment thread tools/validate_repository.py Outdated
Comment thread tools/validate_repository.py Outdated

Copy link
Copy Markdown
Contributor Author

Fixed all three exact-head 2a5713a findings in 26a001b743410e4bf2cbf471c1fddf3e0510a9b3:

  • all 6 linked successful 0.2 audit fixtures now carry the exact generated canonical_plan_json, and the audit field list includes it;
  • the frontend's exact supported plan-contract list and contract-to-schema mapping are checked;
  • all three legacy-output fixture paths require the v0.1 plan contract.

The 0.1 manifests remain byte-for-byte unchanged; no behavioral interpreter or case-ID allowlist was added. Exact-head local validation and push/PR CI pass.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 26a001b743410e4bf2cbf471c1fddf3e0510a9b3. All prior findings are resolved, published 0.1 manifests remain byte-identical, CI is green, and the repository intentionally contains only generic structural/integrity validation—not behavioral interpretation.

@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: 26a001b743

ℹ️ 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 tools/validate_repository.py Outdated
Comment thread tools/validate_repository.py Outdated
Comment thread tools/validate_repository.py Outdated

Copy link
Copy Markdown
Contributor Author

Fixed all three exact-head 26a001b findings in 75f39944af8d0fc259e99316188c016e0ceeac27:

  • the binding audit schema is pinned to the normative SQL DDL path;
  • every successful 0.1/0.2 frontend fixture must declare exactly one plan-output variant with the matching contract rules;
  • every audited binding fixture must carry paired plan/frontend references before its hash chain is checked.

No behavioral interpreter or case-ID allowlist was added. Local validation and exact-head push/PR CI pass.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 75f39944af8d0fc259e99316188c016e0ceeac27. Prior structural-integrity findings are resolved, 0.1 manifests remain immutable, exact-head CI is green, and no behavioral interpreter is present.

@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: 75f39944af

ℹ️ 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 tools/validate_repository.py Outdated

Copy link
Copy Markdown
Contributor Author

Architecture correction in 774f6d70e73961222752189e02ce798d67b7bc75: the Transformation Plan/frontend integrity checker has been removed entirely instead of extended further. Both behavioral validators are deleted, their import/call is gone, and the dedicated dependency/CI install is removed. This specification repository now keeps normative docs, schemas, and declarative fixtures only; executable validation belongs in implementation/adapter repositories. Accordingly, the pending checker-specific operation_count suggestion is superseded by removal of that checker.

@TonisOrmisson

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: 8f1f750fb3

ℹ️ 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".

@TonisOrmisson
TonisOrmisson marked this pull request as ready for review August 4, 2026 10:12
@TonisOrmisson
TonisOrmisson merged commit 3f983de into main Aug 4, 2026
2 checks passed
@TonisOrmisson
TonisOrmisson deleted the agent/conditional-transformation-0.2 branch August 4, 2026 10:13
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.

1 participant