[codex] Add current worktree command with PR lookup#85
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8308fe054c
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 90f637d8e9
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 73e34cdbfc
ℹ️ 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".
| const sharedSource: SettingsInputSource = mainInputs.shared | ||
| ? { input: mainInputs.shared, root: settingsRoot } | ||
| : { input: currentInputs.shared, root: repoRoot }; |
There was a problem hiding this comment.
When wt runs inside a linked worktree, this branch always picks mainInputs.shared whenever the main worktree has .wt/settings.json, even if the linked worktree has its own .wt/settings.json (for example on a branch that changes issue.*, scripts.*, or baseBranch). That regresses prior behavior by ignoring branch-local shared settings in linked worktrees and can make wt current/wt list/wt new use stale config from the main worktree. The fallback should only use main shared settings when the linked shared file is absent.
Useful? React with 👍 / 👎.
1 participant
What changed
wt currentto show only the current worktree without walking and enriching the full worktree listwt currentonly when PR metadata is not already storedWhy
wt lsis useful for the full catalog, but it is heavier than needed when the goal is just to inspect the current worktree. This adds a fast path for the current worktree and keeps PR lookup scoped to a single branch instead of the whole repo.Impact
wt currentuses stored PR metadata first and falls back to a branch-scoped open PR lookup only when neededValidation
bun run typecheckbun test