Skip to content

feat: add research and optimization spec contracts#231

Merged
Pigbibi merged 10 commits into
mainfrom
codex/quant-strategy-spec-v1
Jul 10, 2026
Merged

feat: add research and optimization spec contracts#231
Pigbibi merged 10 commits into
mainfrom
codex/quant-strategy-spec-v1

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add versioned research_spec.v1 and optimization_spec.v1 JSON schemas
  • add dependency-free lifecycle validators and a CLI for future evidence-gate use
  • document v1 invariants and add focused acceptance/rejection coverage

Validation

  • PYTHONPATH=src python3 -m pytest -q tests/test_strategy_spec_validation.py tests/test_lifecycle_contracts.py tests/test_strategy_evidence_package_validator.py tests/test_validate_strategy_evidence_package.py
  • PYTHONPATH=src python3 -m ruff check src/quant_platform_kit/strategy_lifecycle/__init__.py src/quant_platform_kit/strategy_lifecycle/spec_validation.py scripts/validate_strategy_spec.py tests/test_strategy_spec_validation.py
  • git diff --check

Compatibility

This adds optional evidence artifacts and does not alter existing lifecycle or runtime interfaces.

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

Merge allowed: No issues found


Review by Codex PR Review bot • PR

@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: 25a7944034

ℹ️ 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 schemas/research-spec.v1.schema.json Outdated
Comment thread src/quant_platform_kit/schemas/optimization-spec.v1.schema.json
Comment thread schemas/optimization-spec.v1.schema.json Outdated
Comment thread src/quant_platform_kit/strategy_lifecycle/spec_validation.py Outdated
Comment thread src/quant_platform_kit/strategy_lifecycle/spec_validation.py
Comment thread src/quant_platform_kit/strategy_spec/validation.py
Comment thread src/quant_platform_kit/schemas/research-spec.v1.schema.json
Comment thread src/quant_platform_kit/strategy_spec/validation.py
@Pigbibi Pigbibi force-pushed the codex/quant-strategy-spec-v1 branch from f6bf662 to 3618e02 Compare July 10, 2026 20:04

@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: 1a4fb9f0b2

ℹ️ 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/quant_platform_kit/schemas/optimization-spec.v1.schema.json
Comment thread src/quant_platform_kit/schemas/optimization-spec.v1.schema.json
Comment thread src/quant_platform_kit/schemas/research-spec.v1.schema.json
Comment thread docs/quant_strategy_spec_contract.md Outdated
@chatgpt-codex-connector

This comment has been minimized.

@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: 84e281b231

ℹ️ 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/quant_platform_kit/strategy_lifecycle/spec_validation.py Outdated
Comment thread src/quant_platform_kit/strategy_lifecycle/spec_validation.py Outdated
Comment thread src/quant_platform_kit/strategy_lifecycle/spec_validation.py Outdated
@Pigbibi Pigbibi force-pushed the codex/quant-strategy-spec-v1 branch from 0b93997 to 69b33f8 Compare July 10, 2026 22:40

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

ℹ️ 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/quant_platform_kit/strategy_spec/validation.py Outdated
Pigbibi and others added 9 commits July 11, 2026 06:55
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi Pigbibi force-pushed the codex/quant-strategy-spec-v1 branch from cb8e15b to 8b79647 Compare July 10, 2026 22:55
@Pigbibi Pigbibi enabled auto-merge July 10, 2026 22:59
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi Pigbibi merged commit 9bb8f31 into main Jul 10, 2026
3 checks passed
@Pigbibi Pigbibi deleted the codex/quant-strategy-spec-v1 branch July 10, 2026 23:04
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