Skip to content

feat: add group colors and harden Grok Build tool calling #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
WilliamWang1721 wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

feat: add group colors and harden Grok Build tool calling #20

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e31f95d
chore: bootstrap feature branch
WilliamWang1721 Jul 12, 2026
d656ad8
docs: add Stage 12 PR delivery manifest
WilliamWang1721 Jul 12, 2026
5bff78d
chore: remove temporary PR bootstrap
WilliamWang1721 Jul 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions docs/reviews/GROUP_COLOR_GROK_TOOLCALL_PR_MANIFEST.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
# Group Color and Grok Build Tool Calling PR Manifest

This draft tracks the reviewed Stage 12 implementation for:

- Persistent custom group colors with strict `#RRGGBB` validation.
- Stable automatic fallback colors for existing groups.
- Group selector layout fixes so protocol badges no longer obscure group names.
- Grok Build tool-calling compatibility fixes for `tool_choice`, sequential tool calls, argument completion, terminal response snapshots, sparse output indexes, and duplicate terminal events.
- A dedicated Grok Tool Calling account probe.
- Migration `161_add_group_color.sql` and Ent runtime descriptor alignment.

## Reviewed source archive

- Google Drive: https://drive.google.com/file/d/1NqJAVDm61KlMZh76TKScGi6FQqez0BiQ/view?usp=drivesdk
- Local artifact: `LightBridge-0.2.80-group-color-grok-toolcall-stage12-full.zip`

The archive was created after repository-level static checks and is retained as the source-of-truth payload for this draft.

## Validation completed in the sandbox

- Go formatting and AST parsing.
- Frontend TypeScript/Vue syntax parsing.
- YAML/JSON parsing and duplicate-key checks.
- GitHub Actions shell syntax checks.
- Release configuration invariant checks.
- Codebase inventory validation.
- Secret scan.
- ZIP CRC/integrity validation.

Full Go 1.26.5 compilation, pnpm build, and real Grok Build E2E tool-calling tests remain required before this PR can be marked ready for merge.

## Important delivery note

The ChatGPT GitHub connector cannot upload a sandbox ZIP as a repository tree in one operation. This draft is intentionally not presented as merge-ready until the reviewed payload is applied to the branch without overwriting newer `main` changes.
Loading