Skip to content

fix(hq-ui): Giao việc groups sessions like the sidebar, disambiguates duplicate names (v0.92.1) - #42

Merged
8syncdev merged 2 commits into
mainfrom
fix/assign-groups
Oct 15, 2026
Merged

8syncdev merged 2 commits into
mainfrom
fix/assign-groups

Conversation

@8syncdev

Copy link
Copy Markdown
Contributor

Why

On live data every owner-started session fell under "Phiên khác" in the new Giao việc dialog (the sidebar groups them under "8sync-startup ⎇ main"), and duplicate names (two "rd" rooms, two "web" rooms) were indistinguishable.

What

  • lib/assign.ts: groupTargets(sessions, projects, rooms) — registered projects first, then git folders from /api/office rooms ("<name> ⎇ <branch>", case-insensitive root), then "Phiên khác". distinctLabels() adds · <8 id chars> to duplicate labels.
  • assign-dialog.tsx: takes rooms; list rows, fan-out chips and the summary line use the distinct label.
  • hq-app.tsx: passes office.office?.rooms.
  • Tests: grouping by room folder, no-room last, distinctLabels.
  • Version 0.92.1 + CHANGELOG.

Verified

  • pnpm vitest run tests/assign-ui.test.ts 11/11, tsc --noEmit, eslint clean (local, light).
  • Preview screenshot with two stub "rd" sessions shows "rd · 11111111" and "rd · 22222222".
  • Not verified: Windows/macOS browsers (CI covers the Rust build on all three OS).

🤖 Generated with Claude Code

8syncdev and others added 2 commits October 15, 2026 16:18
…tes duplicate names

Sessions started outside a registered project are grouped by the git folder
their office room reports ("su-code ⎇ main"), as the sidebar does, instead
of all landing under "Phiên khác". Two live sessions with the same name
show " · <8 id chars>" in the list, the fan-out chips and the summary line
so the owner can tell which one a task goes to.

Co-Authored-By: Claude Fable 5.1 <[email protected]>
@8syncdev
8syncdev merged commit fc12879 into main Oct 15, 2026
6 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.

1 participant