chore(release): v2.6.1 #26
Conversation
There was a problem hiding this comment.
Pull Request Overview
This release implements assistant alias support across the application, allowing users to customize how Claude and Codex are displayed in the UI through CLI flags or environment variables. The feature maintains backward compatibility while adding personalization options.
- Added assistant alias configuration via CLI flags (
--claude-alias,--codex-alias) and environment variables - Updated UI components to display configured aliases instead of hardcoded names
- Added comprehensive test coverage for the new alias functionality
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| test/server-alias.test.js | New test file validating server alias configuration and defaults |
| src/server.js | Added alias configuration and API endpoint to serve aliases to frontend |
| src/public/session-manager.js | Updated notification messages to use configured aliases |
| src/public/app.js | Added alias loading, UI updates, and dynamic text replacement |
| package.json | Version bump to 2.6.1 |
| README.md | Documentation for new alias feature with usage examples |
| CHANGELOG.md | Release notes documenting the new alias functionality |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| ``` | ||
| These aliases are for display purposes only; they do not change which underlying CLI is launched. | ||
| ``` |
There was a problem hiding this comment.
The code block is not properly closed. There's an extra closing triple backtick that creates malformed markdown structure.
![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 suggestions.
Reply with @codex fix comments to fix any unresolved comments.
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, or 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 fix this CI failure" or "@codex address that feedback".
| it('should set aliases from options', function() { | ||
| const server = new ClaudeCodeWebServer({ | ||
| claudeAlias: 'Buddy', | ||
| codexAlias: 'Robo', | ||
| noAuth: true // avoid auth middleware complexity |
There was a problem hiding this comment.
[P1] Tear down ClaudeCodeWebServer in tests to prevent hanging suite
Both alias tests instantiate ClaudeCodeWebServer and immediately assert on server.aliases, but the constructor schedules a 30 s setInterval in setupAutoSave() and registers process listeners. Because the test never calls handleShutdown()/close() or clears the interval, npm test never exits—the event loop stays alive until the process is killed (confirmed by the suite hanging when run locally). Add an afterEach that invokes the server’s shutdown path to clean up timers.
Useful? React with 👍 / 👎.
What
Release v2.6.1
Highlights
--claude-alias <name>,--codex-alias <name>CLAUDE_ALIAS,CODEX_ALIAS/api/confignow returnsaliasesfor the frontendtest/server-alias.test.jsNotes
Please review and merge. After merge, tag v2.6.1 already exists and a GitHub release will be available.