feat: add /btw background session command#18917
Conversation
|
The following comment was made by an LLM, it may be inaccurate: Potential Duplicates Found:
Summary: PR #18917 is explicitly split from PR #18635 (as mentioned in the description), so #18635 is the original parent PR. PR #17198 appears to be an earlier attempt at the same feature. These should be reviewed to ensure no duplicate work is being done across all three PRs. |
|
@atharvau This is the split-out I separated the tool-limit work into #18916, so this PR is just the I also re-ran the local verification here and it passes:
If this scope looks good now, do you think this is ready to merge? |
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
f534a81 to
e47f5fa
Compare
|
Closing this PR after self-review: this feature (background session command) falls under core product functionality and did not go through a design review with the core team as required by the contributing guidelines. |
Issue for this PR
Closes #16992
Split from #18635
Type of change
What does this PR do?
This PR contains the
/btwcommand work split out from #18635.It includes:
/btw ask,/btw todo, and/btw status/btw-specific synthetic message filtering so status/result notes do not leak back into later model turns/btwflow into smaller helpersHow did you verify your code works?
cd packages/opencode && bun typecheckcd packages/opencode && bun test test/session/btw.test.ts test/session/message-v2.test.tsScreenshots / recordings
N/A - backend/CLI command, no UI changes.
Checklist