feat: /later command + workspace directory convention#67
Conversation
- /later <idea> — saves to ~/.roundhouse/workspace/later.md with date - /later (no args) — shows saved items - tools.md: all agent work output goes under ~/.roundhouse/workspace/ - Registered in Telegram bot command menu
![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: a1fcf5ede5
ℹ️ 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".
|
|
||
| export async function handleLater(ctx: LaterCommandContext): Promise<void> { | ||
| const { thread, text, postWithFallback } = ctx; | ||
| const idea = text.replace(/^\/later\s*/i, "").trim(); |
There was a problem hiding this comment.
The idea extraction regex only strips /later followed by whitespace, so Telegram group-style commands like /later@MyBot or /later@MyBot buy milk are parsed incorrectly: the first case is treated as a new idea @MyBot instead of showing the list, and the second saves @MyBot buy milk instead of just the note text. Since command routing already accepts @botname variants, this handler should strip the optional @... suffix as part of command parsing to avoid corrupting saved entries in group chats.
Useful? React with 👍 / 👎.
1. Escape </persona> in persona content (XML injection prevention) 2. mtime check uses !== instead of > (catches deletions/replacements) 3. mkdirSync before writeSettings (handles fresh installs) 4. /later@BotName suffix stripped in group chats All from Codex PR review comments on #75, #77, #66, #67. 393 tests green. Co-authored-by: Loki FastStart <loki@faststart.internal>
1 participant
Two changes:
/latercommand — Quick-capture ideas from Telegram/later buy more coffee→ saves to~/.roundhouse/workspace/later.mdwith date/later(no args) → shows all saved itemsWorkspace directory convention in tools.md
~/.roundhouse/workspace/~/with random agent artifacts393 tests green, no regressions.