Skip to content

docs(skills): require campaign asset cleanup - #2263

Merged
samchon merged 1 commit into
masterfrom
docs/campaign-cleanup-assets
Jul 20, 2026
Merged

docs(skills): require campaign asset cleanup#2263
samchon merged 1 commit into
masterfrom
docs/campaign-cleanup-assets

Conversation

@samchon

@samchon samchon commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Intent

Require every issue and benchmark campaign assignment to remove its disposable worktree and assigned mutable assets, including Go temporary caches, before completion.

Scope

  • Record and remove every assigned campaign mutable root before removing a finished issue-campaign worktree.
  • Apply the same completion rule to verifier and mutation worktrees.
  • Define benchmark-campaign cleanup for disposable worktrees, mutable roots, Go temporary assets, and any associated disposable local branch.

Deferred

This documentation-only pull request does not make #2262 ready, alter its prospective gate, or merge any campaign pull request.

Local verification

  • Re-read the complete .agents/skills/issue-campaign/development.md and .agents/skills/benchmark/SKILL.md contexts after editing.
  • git diff --check
  • git show --check --stat 22c6c8ca67ffd67e496938c64af2ec1de3c6b907

Campaign Actions are deliberately cancelled; the exact-SHA cancellation records are maintained in the campaign knowledge base.

@samchon

samchon commented Jul 20, 2026

Copy link
Copy Markdown
Owner Author

Self-review opening for 22c6c8ca67ffd67e496938c64af2ec1de3c6b907.

I will review the complete base-to-head diff and both full SKILL contexts. The boundary check covers an isolated mutable root with no worktree, detached worktrees with no topic branch, disposable topic branches, Go temporary assets, retained evidence, and the required order of removal and verification. I will report remediation before changing anything; this PR remains draft and is not a merge action.

@samchon

samchon commented Jul 20, 2026

Copy link
Copy Markdown
Owner Author

Self-review complete: no remediation.

Reviewed immutable master...22c6c8ca67ffd67e496938c64af2ec1de3c6b907 (two SKILL files only) and re-read both complete documents. The cleanup rule preserves compact evidence before destructive removal, removes every assigned GOCACHE, GOTMPDIR, TTSC_CACHE_DIR, generated/consumer/staging root, then proves their absence. It covers verifier and mutation worktrees.

Boundary results:

  • An isolated measurement root without a worktree removes and verifies its roots; worktree removal is explicitly conditional.
  • A detached worktree has no associated local topic branch to delete.
  • A worktree with an associated disposable topic branch removes that branch only after worktree removal and pruning.

git diff --check master...HEAD passed. No product, package, generated artifact, consumer, or workflow surface changed; no pnpm format or build/test applies to this active-campaign Markdown-only rule change. PR remains draft; no merge action was taken.

@samchon
samchon marked this pull request as ready for review July 20, 2026 12:06
@samchon
samchon merged commit da88f53 into master Jul 20, 2026
2 of 3 checks passed
@samchon
samchon deleted the docs/campaign-cleanup-assets branch July 20, 2026 12:06
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