Skip to content

fix(land): make tmp file cleanup mandatory #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
eumaninho54 wants to merge 1 commit into from
Closed

fix(land): make tmp file cleanup mandatory #26

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions src/commands/land/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,20 +39,23 @@ Instruct the commit agent to:
- If yes: read its contents as the diff instead of running `git diff HEAD`
- If no: run `git diff HEAD` normally

### 4. Delete temp file
### 4. Delete temp file — MANDATORY

**This step is non-optional. Run it unconditionally, even if /commit failed or was skipped.**

After /commit completes (success or failure), the orchestrator deletes the temp file:
```bash
rm .claude/.aiworkers-context.tmp
rm -f .claude/.aiworkers-context.tmp
```

Do not proceed to step 5 without confirming this command ran.

### 5. Run /pr (if requested)

If the user's original request included creating a PR, read and follow `.claude/skills/aiworkers/pr/SKILL.md`.

## Rules

- The orchestrator is solely responsible for saving and deleting `.claude/.aiworkers-context.tmp` — skills only read it, never delete it
- Always delete `.claude/.aiworkers-context.tmp` after /commit, even if commit fails
- **Always** delete `.claude/.aiworkers-context.tmp` after /commit, even if commit fails — use `rm -f` so it never errors
- If /branch fails, delete the tmp file before stopping
- Never push the branch unless /pr is being run (which handles pushing itself)
Loading