Skip to content

feat: update session thread root content once started#7

Merged
0xmrpeter merged 1 commit into
Open-ACP:mainfrom
kevin-david:thread-root
May 11, 2026
Merged

feat: update session thread root content once started#7
0xmrpeter merged 1 commit into
Open-ACP:mainfrom
kevin-david:thread-root

Conversation

@kevin-david

@kevin-david kevin-david commented May 7, 2026

Copy link
Copy Markdown
Contributor

Replace the placeholder message with session details, enhancing user experience.

Tested locally in combination with #6

Replace the placeholder message with  session details, enhancing user experience.
@0xmrpeter

Copy link
Copy Markdown
Contributor

Nice UX improvement — the stale "⏳ Setting up..." placeholder has been a papercut for a while. The updateSessionThreadStarter implementation is clean and follows the forums.ts patterns well (defensive try/catch, log.warn on failure, consistent with renameSessionThread and deleteSessionThread).

One minor note: the format string 📂 **${agentName}** — `${workingDirectory}` is duplicated verbatim in both new-session.ts and adapter.ts. If the format ever changes, both sites need updating — a small shared helper exported from forums.ts (e.g. buildSessionStarterContent) would keep it in one place.

No blocking issues — good to go!

@0xmrpeter 0xmrpeter merged commit fe92041 into Open-ACP:main May 11, 2026
3 checks passed
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.

2 participants