Skip to content

fix(runtime): preserve inherited run bundle ownership - #2080

Merged
codeforester merged 1 commit into
mainfrom
bug/2069-20260902-prevent-delegated-base-children-from-finalizing-inherited-ru
Sep 3, 2026
Merged

codeforester merged 1 commit into
mainfrom
bug/2069-20260902-prevent-delegated-base-children-from-finalizing-inherited-ru

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

Prevent delegated Base child invocations from finalizing an inherited parent run bundle. This removes repeated invalid-context warnings during workspace fan-out while preserving shared logging and history ownership.

Issue

Fixes #2069

Validation

  • bats cli/bash/commands/basectl/tests/runtime-dispatch.bats — 40 passed
  • bats cli/bash/commands/basectl/tests/workspace.bats — 15 passed
  • BASE_CLI_SOURCE_DIR=/Users/rameshhp/work/base-cli/lib/python PYTHONPATH=/Users/rameshhp/work/base-cli/lib/python:lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest cli/python/base_projects/tests/test_workspace_clone.py -q — 5 passed
  • env -u BASE_HOME BASE_BASH_LIBS_DIR=/Users/rameshhp/work/base-bash-libs/lib/bash BASE_CLI_SOURCE_DIR=/Users/rameshhp/work/base-cli/lib/python BASE_CACHE_DIR=/private/tmp/base-test-run-bundle-2069 ./bin/base-test — 1115 Python and 899 BATS passed
  • ShellCheck and git diff --check passed

Demo Impact

None.

Notes

Only the invocation that creates a run bundle finalizes it. Inherited children validate the bundle but leave its lifecycle to the parent. No workspace clone, manifest, authentication, or GitHub API semantics change.

@codeforester
codeforester requested a review from a team as a code owner September 2, 2026 18:17
@codeforester
codeforester merged commit fb8f79e into main Sep 3, 2026
19 checks passed
@codeforester
codeforester deleted the bug/2069-20260902-prevent-delegated-base-children-from-finalizing-inherited-ru branch September 3, 2026 07:32
codeforester added a commit that referenced this pull request Sep 3, 2026
## Summary

- Keep completion history ownership with the invocation that created the
run bundle.
- Prevent inherited Base children from marking the parent bundle
complete.
- Clarify the delegated-history contract and add regression coverage.

## Issue

Fixes #2083

Follow-up to #2080.

## Validation

- `bats --print-output-on-failure
cli/bash/commands/basectl/tests/runtime-dispatch.bats` — 40 passed
- `bats --print-output-on-failure
cli/bash/commands/basectl/tests/workspace.bats` — 15 passed
- Workspace clone tests — 5 passed
- Real workspace dry-run fan-out — 0 inherited-context warnings, 1
primary history row, 1 run bundle
- `./bin/base-test` — 1,115 Python and 899 BATS passed
- ShellCheck and `git diff --check` passed

## Docs Impact

Updated `docs/observability.md` to document that inherited Base children
do not write separate completion records or mutate the parent lifecycle.

Workspace clone semantics and repository materialization behavior are
unchanged.
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.

Prevent delegated Base children from finalizing inherited run bundles

1 participant