Skip to content

Remove the migrated Private Lima calibration harness - #325

Merged
toku345 merged 1 commit into
mainfrom
agent/remove-private-lima-calibration
Aug 8, 2026
Merged

Remove the migrated Private Lima calibration harness#325
toku345 merged 1 commit into
mainfrom
agent/remove-private-lima-calibration

Conversation

@toku345

@toku345 toku345 commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the active Private Lima calibration harness and runtime test suite after the destination receipt passed
  • retain the immutable v1/v2 packages, ADR history, and historical design snapshots
  • move the protected-history test into a dedicated suite with full-history CI
  • remove the harness-specific chezmoi ignore entry and Private Lima runtime/profile CI jobs

Why

The active implementation is now maintained in toku345/private-outer-loop. Removing the frozen duplicate restores a focused dotfiles boundary without changing historical inputs or local runtime state.

Validation

  • protected-history suite — 1 test, 0 skipped
  • protected v1/v2 and ADR 0030/0031 — unchanged from the implementation base
  • active harness and runtime test directories — absent
  • git diff --check — passed
  • no __pycache__ or .pyc artifacts

Safety

No calibration CLI, Lima lifecycle command, authentication, model call, state migration, cleanup, or evidence mutation was performed.

Co-authored-by: Codex <noreply@openai.com>
@toku345
toku345 marked this pull request as ready for review August 8, 2026 07:51
@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown

Greptile Summary

The PR removes the migrated Private Lima calibration implementation and its runtime validation while retaining immutable historical inputs and design records.

  • Deletes the active calibration harness and its dedicated tests.
  • Replaces the harness/profile CI jobs with a full-history protected-input test.
  • Moves the protected-history test into a dedicated suite.
  • Removes the obsolete chezmoi ignore entry and marks retained designs as historical snapshots.

Confidence Score: 5/5

The PR appears safe to merge because the migrated implementation is removed cleanly and the retained immutable history remains covered by a passing full-history test.

No surviving caller depends on the deleted harness, the obsolete chezmoi rule is removed with its target directory, and the replacement protected-history suite can resolve its base commit and passes.

Important Files Changed

Filename Overview
.github/workflows/ci.yml Replaces obsolete harness and profile validation jobs with a full-history protected-input suite whose checkout and test command satisfy the retained test's requirements.
tests/outer_loop_history/test_protected_history.py Moves the unchanged protected-history test into a dedicated discovery directory; the implementation-base commit is available and the suite passes.
.chezmoiignore Removes an ignore entry that became obsolete when the corresponding source directory was deleted.
docs/design/codex-claude-outer-loop-private-lima-calibration-implementation.md Labels the retained implementation design as a historical snapshot and points readers to the active external repository.
docs/design/codex-claude-outer-loop-private-lima-calibration.md Labels the retained calibration design as historical and identifies the repository containing the active implementation.
tools/outer-loop-lima-calibration/calibrate.py Deletes the migrated calibration CLI along with its complete supporting implementation and assets.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[dotfiles active calibration harness] -->|migrated| B[private-outer-loop repository]
  A -->|removed from dotfiles| C[Harness code and runtime tests deleted]
  D[Immutable v1/v2 packages and ADRs] --> E[Protected-history test]
  E --> F[Full-history CI job]
  G[Historical design snapshots] --> H[Retained in dotfiles]
Loading

Reviews (1): Last reviewed commit: "Remove the migrated Private Lima calibra..." | Re-trigger Greptile

@toku345
toku345 merged commit 6cd2428 into main Aug 8, 2026
17 checks passed
@toku345
toku345 deleted the agent/remove-private-lima-calibration branch August 8, 2026 07: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.

1 participant