fix #253: rename capability_id to contract_id in skill_executor ledger calls #268

Merged
Mark merged 1 commit from fix/mesh-seed-253-contract-id into master 2026-08-07 14:45:14 +00:00
Owner

MeshLedger.record_skill_run() and record_secret_access() expect contract_id after the Apr 15 refactor (commit 89483d2), but skill_executor.py still passed capability_id. This caused 3 test failures and a runtime TypeError.

Changes:

  • runtime/skill_executor.py: rename capability_id -> contract_id at all 6 ledger call sites (L64, L93, L119, L140, L156, L330)
  • primitives/_muscle/node_provision/run.py: update caller context key (L134)

Verification:

  • pytest tests/unit/test_skill_promotion_and_championship.py: 38/38 passed
  • ruff check: all clean
MeshLedger.record_skill_run() and record_secret_access() expect contract_id after the Apr 15 refactor (commit 89483d2), but skill_executor.py still passed capability_id. This caused 3 test failures and a runtime TypeError. Changes: - runtime/skill_executor.py: rename capability_id -> contract_id at all 6 ledger call sites (L64, L93, L119, L140, L156, L330) - primitives/_muscle/node_provision/run.py: update caller context key (L134) Verification: - pytest tests/unit/test_skill_promotion_and_championship.py: 38/38 passed - ruff check: all clean
fix #253: rename capability_id to contract_id in skill_executor ledger calls
Some checks failed
CI / Lint and Format (pull_request) Failing after 27s
CI / Type Check (pull_request) Successful in 39s
CI / Code Quality Checks (pull_request) Successful in 38s
CI / Unit Tests (pull_request) Failing after 33s
CI / Full Test Suite (pull_request) Has been skipped
CI / Smoke Tests (pull_request) Failing after 28s
CI / Integration Tests (pull_request) Successful in 1m27s
CI / Security Check (pull_request) Failing after 4m8s
c344e5e4a8
- MeshLedger.record_skill_run and record_secret_access expect contract_id.
- Update all 6 call sites in runtime/skill_executor.py.
- Update node_provision caller context key from capability_id to contract_id.
Mark merged commit 3e38d6290e into master 2026-08-07 14:45:14 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Mark/mesh-seed!268
No description provided.