Skip to content

Complete temporary run retention coverage#1727

Merged
codeforester merged 1 commit into
mainfrom
enhancement/1685-20260721-temp-retention-completion
Jul 21, 2026
Merged

Complete temporary run retention coverage#1727
codeforester merged 1 commit into
mainfrom
enhancement/1685-20260721-temp-retention-completion

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • add failed-run coverage for default temp cleanup and explicit retention
  • document the per-run temporary-data contract in the observability guide
  • keep run metadata, history, and the primary log available in both modes

Why

The retention implementation is already on main; this PR completes issue #1685's
remaining acceptance coverage and documentation. Temporary data is diagnostic-only
and should not accumulate after successful or failed commands unless the user
explicitly opts in with --keep-temp (or the documented Python retention setting).

Validation

  • bats cli/bash/commands/basectl/tests/runtime-dispatch.bats — 29 passed
  • BASE_BASH_LIBS_DIR=... env -u BASE_HOME ./bin/base-test — 1043 passed, 1 skipped
  • PYTHONPATH=lib/python python3 -m unittest lib/python/base_cli/tests/test_base_cli.py lib/python/base_cli/tests/test_app_runtime_boundary.py — 49 passed, 14 skipped
  • git diff --check
  • ShellCheck reports only the existing SC2016 informational warnings in the Bats file

Project and AI context

  • Base Project item moved to In Progress; merge/closure should move it to Done.
  • .ai-context/ unchanged: this is completion coverage and documentation for an
    already-shipped runtime contract, with no new product or architecture decision.

Fixes #1685

@codeforester
codeforester requested a review from a team as a code owner July 21, 2026 13:48
@codeforester
codeforester merged commit 7ee0d65 into main Jul 21, 2026
28 checks passed
@codeforester
codeforester deleted the enhancement/1685-20260721-temp-retention-completion branch July 21, 2026 13:52
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.

Make temporary run data opt-in and remove empty tmp scaffolding

1 participant