Skip to content

Add scoped Code Mode foundation - #139

Merged
kortiene merged 3 commits into
mainfrom
scoped-codemode-foundation-internal
Sep 16, 2026
Merged

kortiene merged 3 commits into
mainfrom
scoped-codemode-foundation-internal

Conversation

@kortiene

Copy link
Copy Markdown
Contributor

Summary

  • Add create_codemode_tool() with an explicit immutable capability set instead of discovering all harness tools.
  • Validate capability names, schemas, arguments, outputs, result projections, and exactly-once output(...) behavior.
  • Add nested capability event ancestry and explicit child-tool inheritance policy.
  • Keep the existing exported codemode tool and sandbox behavior unchanged for compatibility.

Testing

  • uv run --python 3.12 pytest tests/harness_sdk -q (104 passed, 1 skipped)
  • make lint
  • git diff --check

Stack

This is the minimal foundation extracted from #119. The independently reviewable hardening follow-up is kortiene#2; it targets this foundation branch in the contributor fork until the foundation can merge.

@kortiene
kortiene requested a review from a team September 16, 2026 10:27
@kortiene

Copy link
Copy Markdown
Contributor Author

Internal-branch replacement for #137. Both PRs point to the exact same reviewed commit, d75f099787ea3d9da2c604fd1ab301fd0985f63d; this replacement allows secret-dependent checks to run from a trusted repository ref.

@arnaugrispg arnaugrispg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved!!

@kortiene
kortiene added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit 4825a4c Sep 16, 2026
11 of 12 checks passed
@kortiene
kortiene deleted the scoped-codemode-foundation-internal branch September 16, 2026 12:54
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.

2 participants