Skip to content

Rectify: Host-Attested Codex Recipe Delivery and Cumulative-Insertion Immunity#4330

Merged
Trecek merged 58 commits into
developfrom
impl-rectify-codex-host-attested-recipe-delivery-20260722-094943
Jul 23, 2026
Merged

Rectify: Host-Attested Codex Recipe Delivery and Cumulative-Insertion Immunity#4330
Trecek merged 58 commits into
developfrom
impl-rectify-codex-host-attested-recipe-delivery-20260722-094943

Conversation

@Trecek

@Trecek Trecek commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Codex recipe delivery currently treats four different quantities as though they were one limit:

  • the 195,000-byte measured open_kitchen / load_recipe source exemption;
  • Code Mode's 10,000-token default when an outer maximum is omitted;
  • the 56,750-token damage-bounded maximum that can be explicitly requested for the measured recipe;
  • tool_output_token_limit, which controls later history retention rather than the outer Code Mode result.

Implementation Plan

Plan file: /home/talon/projects/generic_automation_mcp/.autoskillit/temp/rectify/rectify_codex_host_attested_recipe_delivery_2026-07-22_094943.md

🤖 Generated with Claude Code via AutoSkillit

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: approved_with_comments

Comment thread tests/server/test_tools_recipe_pull.py Outdated
Comment thread tests/contracts/test_delivery_bound_fitness.py
Comment thread src/autoskillit/server/tools/tools_recipe.py
Comment thread src/autoskillit/execution/backends/_codex_config.py
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated
Comment thread src/autoskillit/execution/backends/_codex_recipe_delivery.py Outdated
Comment thread src/autoskillit/core/types/_type_constants_registries.py Outdated
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated
Comment thread src/autoskillit/execution/backends/_codex_recipe_delivery.py
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: changes_requested

Comment thread src/autoskillit/execution/backends/_codex_recipe_delivery.py Outdated
Comment thread src/autoskillit/execution/backends/_codex_recipe_delivery.py Outdated
Comment thread src/autoskillit/server/_notify.py
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated
Comment thread tests/infra/test_pretty_output_recipe.py

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit review: blocking findings detected. See inline comments — changes required. The formal request-changes event is unavailable because this PR is self-authored.

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit PR Review — Verdict: changes_requested

Comment thread src/autoskillit/server/_recipe_delivery.py
Comment thread src/autoskillit/server/_recipe_delivery.py
Comment thread src/autoskillit/server/_recipe_delivery.py
Comment thread src/autoskillit/server/_recipe_delivery.py Outdated
Comment thread src/autoskillit/execution/backends/_codex_config.py Outdated
Comment thread src/autoskillit/server/tools/tools_recipe.py Outdated
Comment thread tests/server/_helpers.py
Comment thread tests/execution/backends/test_codex_recipe_delivery.py
Comment thread tests/execution/backends/test_codex_recipe_delivery_conformance.py
Comment thread tests/server/test_tools_recipe_pull.py Outdated

@Trecek Trecek left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

AutoSkillit review: blocking findings detected. See inline comments — changes required. GitHub does not permit the PR author to submit a REQUEST_CHANGES review on their own PR, so this summary is posted as a comment.

@Trecek
Trecek added this pull request to the merge queue Jul 23, 2026
Merged via the queue into develop with commit c1d5bc2 Jul 23, 2026
3 checks passed
@Trecek
Trecek deleted the impl-rectify-codex-host-attested-recipe-delivery-20260722-094943 branch July 23, 2026 03:30
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