Skip to content

Recover minimal catalog and Dolt lifecycle integration - #12

Draft
TonisOrmisson wants to merge 119 commits into
mainfrom
agent/catalog-dolt-lifecycle-minimal
Draft

Recover minimal catalog and Dolt lifecycle integration#12
TonisOrmisson wants to merge 119 commits into
mainfrom
agent/catalog-dolt-lifecycle-minimal

Conversation

@TonisOrmisson

@TonisOrmisson TonisOrmisson commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Status

Draft pending publication of openstatspec-specification==0.2.1.

This branch rebuilds the explicit catalog and Dolt lifecycle slice from current v0.4 main without the older mixed recovery commit.

Included scope

  • explicit catalog initialization and fail-closed no-auto-create gates
  • exact Dolt identity/declaration binding and read-only working-set evidence
  • Dolt operational writes rejected before mutation while the concrete declaration set is empty
  • catalog shape/bijection verification, bounded cleanup, and export audit recovery
  • packaged openstatspec-specification==0.2.1 dependency pinned to commit 5b62bce1d2f4d719ac6ca42d73f07e7a127c7093
  • focused catalog lifecycle and conformance tests

Invariants

  • the same logical dataset and physical wide table remain in place
  • no persistent staging, copy, snapshot, rollback, or parallel history layer
  • Dolt remains the sole history/versioning layer
  • core code does not call DOLT_ADD, DOLT_COMMIT, checkout, or reset
  • v0.4 transformation CLI/API surfaces remain present

Current evidence

  • GitHub Actions run #317: 14/14 jobs green on exact head against the final 0.2.1 package commit
  • Python 3.11, 3.12, 3.13, and 3.14 non-service suites green
  • package build/install smoke green
  • PostgreSQL 17.10/18.4 green
  • MySQL 8.4.11/9.7.2 green
  • MariaDB 11.4.12/11.8.8/12.3.2 green
  • Dolt 2.2.2/2.2.3 read-only and fail-closed/no-mutation gates green
  • raw Dolt limit probing is isolated as non-claiming candidate_evidence

Remaining gate

Do not mark ready or merge until specification PR #6 is merged, package 0.2.1 is published, and the final immutable specification commit/package installation is reverified.

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: f18e02bd00

ℹ️ 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 src/openstatspec/sql/wide.py Outdated
Comment thread src/openstatspec/sql/wide.py

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: 04f86fa8be

ℹ️ 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 src/openstatspec/sql/wide.py

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: 6d7e3e2f1d

ℹ️ 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 src/openstatspec/sql/wide.py
Comment thread src/openstatspec/spss/sav.py

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: f17b3b277b

ℹ️ 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 src/openstatspec/spss/sav.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: 0b1e891891

ℹ️ 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 src/openstatspec/sql/profiles.py Outdated
Comment thread src/openstatspec/spss/sav.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: cceff0605c

ℹ️ 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 src/openstatspec/sql/wide.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: 0713ca79e3

ℹ️ 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 src/openstatspec/sql/wide.py
Comment thread src/openstatspec/sql/wide.py

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: e98ef83eb9

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

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