Improve Linear routing help/status and add concrete recovery guidance#26
Conversation
|
…,-error,-and-help-ou
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 64574f13d2
ℹ️ 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
- 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 address that feedback".
| lines: [ | ||
| "Profiles are global Linear auth identities.", | ||
| "Repos can bind a default Linear project route plus optional linked projects.", | ||
| "When you omit --profile, Hack prefers the repo binding first and then the global default profile.", |
There was a problem hiding this comment.
Correct the help text about repo-vs-global profile precedence
For repos that bind only a default project/team and do not set controlPlane.routing.overrides.linear.profile, this sentence is inaccurate. resolveLinearProfileSelection() in src/control-plane/extensions/linear/auth.ts:785-799 only prefers the repo-level profile override; otherwise it falls back to the global default profile. As written, the new help text implies any repo binding wins, which can send users debugging the wrong setting and lead them to run syncs against the wrong Linear account.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
ok can you put up a pr to fix this please
## 1.20.0 (2026-03-24) * feat(codex): add slim install path for managed containers ([38f6b78](38f6b78)) * improve linear routing status output ([64574f1](64574f1)) * Merge branch 'main' into codex/linear-mention-hack-444-improve-status,-error,-and-help-ou ([7c08dd5](7c08dd5)) * Merge pull request #26 from hack-dance/codex/linear-mention-hack-444-improve-status,-error,-and-help ([b88c8b7](b88c8b7)), closes [#26](#26) * docs(agents): require conventional commits for release flow ([4a929c3](4a929c3))
Motivation
Description
Linear routing modelhelp panel inrenderHelpforhack linearthat explains profile vs repo binding precedence.hack linear statusoutput viarenderLinearStatusPayloadto include bound profile, default project, default team, and linked project count using a newdescribeLinearBindingStatehelper.resolveLinearAuthSettingsResultto include the selection source and a concrete recovery command tailored to--profile, project routing, global default, or implicit default cases.hack linear projects,hack linear project-bind, orhack linear statusfor recovery.T-RJJKTFR97Tto track pre-existing cognitive-complexity lint issues in the Linear extension rather than changing large functions now.Testing
bun test tests/linear-commands.test.tsand all Linear command unit tests passed.bun test tests/github-auth.test.tsand all GitHub auth tests passed.bun test tests/linear-commands.test.ts tests/github-auth.test.tsand they all passed.bunx ultracite checkwhich reported existing cognitive-complexity warnings in Linear auth/command functions; a follow-up ticketT-RJJKTFR97Twas filed to address that debt.Codex Task