Skip to content

chore(hooks): run desktop typecheck in pre-push - #5110

Merged
wpfleger96 merged 1 commit into
mainfrom
wpfleger/pre-push-desktop-typecheck
Aug 6, 2026
Merged

chore(hooks): run desktop typecheck in pre-push#5110
wpfleger96 merged 1 commit into
mainfrom
wpfleger/pre-push-desktop-typecheck

Conversation

@wpfleger96

Copy link
Copy Markdown
Member

The local pre-push gate ran biome (desktop-check) and node:test (desktop-test) for desktop changes but never tsc, so TypeScript errors surface no earlier than CI's desktop-core job (just desktop-build = tsc && vite build). A branch with type errors passes every local hook today.

This adds a desktop-typecheck pre-push command running just desktop-typecheck (tsc --noEmit) with the same glob/exclude as desktop-check, and updates the hook documentation in AGENTS.md. CI is unchanged — it already typechecks via desktop-build.

Pre-push only ran biome and node:test for desktop, so TypeScript errors
surfaced no earlier than CI's desktop-build (tsc && vite build). Gate
tsc --noEmit locally with the same glob as desktop-check.

Signed-off-by: Will Pfleger <pfleger.will@gmail.com>
@wpfleger96
wpfleger96 requested a review from a team as a code owner August 6, 2026 20:55
@wpfleger96
wpfleger96 merged commit c777d4f into main Aug 6, 2026
3 checks passed
@wpfleger96
wpfleger96 deleted the wpfleger/pre-push-desktop-typecheck branch August 6, 2026 20:57
wpfleger96 pushed a commit that referenced this pull request Aug 6, 2026
* origin/main:
  Alert community owners and admins when a new key joins (#4900)
  fix(desktop): prevent sidebar prefs from reverting on stale-localStorage boot (#5086)
  chore(hooks): run desktop typecheck in pre-push (#5110)
  feat(identity): recover desktop identity from a signed-in phone (#4845)

Signed-off-by: Duncan <dcfd242e557282d7a1e2cf2e6877522682f1e5c6156dc92ca7d90eaedd3b0f95@buzz.block.builderlab.xyz>
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