Skip to content

feat(codebuddy-hud): display session name from /rename in status line - #1

Open
lyydsheep wants to merge 2 commits into
tyanxie:mainfrom
lyydsheep:feat/codebuddy-hud-session-name
Open

feat(codebuddy-hud): display session name from /rename in status line#1
lyydsheep wants to merge 2 commits into
tyanxie:mainfrom
lyydsheep:feat/codebuddy-hud-session-name

Conversation

@lyydsheep

Copy link
Copy Markdown
  • Add sessionName field to TranscriptData interface
  • Parse ai-title and custom-title entries from transcript to extract session name set by /rename command or AI auto-generated title
  • Update renderIdentityLine to show session name with brightCyan color when available, falling back to dimmed #sessionId
  • Support CodeBuddy Code transcript format: parse top-level function_call/function_call_result entries for tool/agent/task tracking
  • Add getSessionId helper to stdin utilities
  • Add sessionId display option to default config

- Add sessionName field to TranscriptData interface
- Parse ai-title and custom-title entries from transcript to extract
  session name set by /rename command or AI auto-generated title
- Update renderIdentityLine to show session name with brightCyan color
  when available, falling back to dimmed #sessionId
- Support CodeBuddy Code transcript format: parse top-level
  function_call/function_call_result entries for tool/agent/task tracking
- Add getSessionId helper to stdin utilities
- Add sessionId display option to default config
@tyanxie

tyanxie commented May 6, 2026

Copy link
Copy Markdown
Owner

Hi, thanks for the contribution! The session name feature is a great idea.

However, I noticed that pnpm build fails on this branch. After investigating, it appears that some changes were made directly to the dist/ directory (the compiled output) rather than to the src/ source files.

In this project, dist/ is auto-generated from src/ by the TypeScript compiler (pnpm build). You should not edit dist/ files directly.

Specifically, the getSessionId function and the sessionId display config option were added in dist/stdin.js and dist/config.js, but their corresponding source definitions in src/stdin.ts, src/config.ts, and src/types.ts are missing. Since src/render/identity.ts already references these, the TypeScript compiler throws errors about non-existent exports and properties.

I've just set up CI to enforce this going forward — PR checks will now verify that pnpm build passes and that dist/ is not manually modified. After a PR is merged, CI will automatically rebuild and commit dist/.

So please update the source files, make sure pnpm build passes locally, and remove any dist/ changes from your commits.

Thanks!

@lyydsheep

Copy link
Copy Markdown
Author

Fixed in a0e80c8.

  • Moved the session ID config/type/helper and CodeBuddy transcript parsing into src/.
  • Restored all dist/ files to the base branch so the PR is source-only.
  • Verified with pnpm build and pnpm test (131 tests passing).

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