Skip to content

task-observer: new-session updates from 2026-06-17#58

Open
divad12 wants to merge 1 commit into
masterfrom
task-observer/new-session-2026-06-17
Open

task-observer: new-session updates from 2026-06-17#58
divad12 wants to merge 1 commit into
masterfrom
task-observer/new-session-2026-06-17

Conversation

@divad12

@divad12 divad12 commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Target\nskill: .agents/skills/new-session/SKILL.md\n\n## Observations addressed\n- Observation 12 (journology/log.md): Reuse existing clean worktrees before creating another → changed: added "Worktree Recovery" section describing the 4-step process: stash/commit main-checkout work, inspect git worktree list --porcelain, fast-forward a clean existing worktree if available, only create a new worktree as a last resort.


Generated by Claude Code

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8524d5d572

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


When work has accidentally accumulated in the main checkout and needs to move to a worktree:

1. Commit or stash the main-checkout changes on the correct branch.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Apply stashed changes in the recovered worktree

If the stash option here is used, the recovery steps never apply or pop that stash in the candidate/new worktree; the later fast-forward step only moves committed branch refs. In the case where main has uncommitted work that gets stashed, the selected worktree will still be clean and the actual changes remain hidden in the main checkout's stash, making the recovery appear to have lost the work. Either require committing before reuse or add an explicit git stash apply/pop step in the destination worktree.

Useful? React with 👍 / 👎.

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