Skip to content

Fix GitHub auth lint failures and harden CI setup#172

Merged
skulidropek merged 1 commit intomainfrom
fix/ci-check-github-token-validation
Mar 21, 2026
Merged

Fix GitHub auth lint failures and harden CI setup#172
skulidropek merged 1 commit intomainfrom
fix/ci-check-github-token-validation

Conversation

@skulidropek
Copy link
Contributor

Summary

  • replace the GitHub user decode path with Schema.parseJson(...) so token validation no longer uses a forbidden unknown boundary type
  • refactor GitHub auth status aggregation to avoid callback-reference lint violations and reduce nested inline logic
  • keep the status report behavior unchanged while making the lib lint step pass again
  • update the shared GitHub Actions setup to use pnpm/action-setup@v5
  • switch the shared dependency install step to pnpm install --frozen-lockfile so Check and dependency validation run with the same install mode

Verification

  • pnpm install --frozen-lockfile
  • pnpm --filter ./packages/app check
  • pnpm --filter ./packages/lib typecheck
  • pnpm --filter ./packages/lib exec eslint src/usecases/auth-github.ts src/usecases/github-token-validation.ts --ext .ts

@skulidropek skulidropek merged commit 542d8c7 into main Mar 21, 2026
12 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