Skip to content

Conversation

@thoroc
Copy link
Contributor

@thoroc thoroc commented Jan 31, 2026

Summary

This PR establishes the foundation for Biome-based linting and formatting that all subsequent PRs will use.

Changes

  • Add Biome v2.3.13 configuration with custom rules
  • Configure lint, format, and format:check targets in nx.json
  • Add validate-tsdoc executor for documentation validation
  • Add typecheck executor for TypeScript validation
  • Update lefthook.yml with Biome pre-commit hooks
  • Update validate-pr.yml workflow for Biome
  • Update generators to include Biome configuration

Dependencies

  • None - This is the foundation PR

Blocks

Testing

  • Biome lint passes: bun run lint
  • Biome format check: bun run format:check
  • TSDoc validation works: bunx nx run-many --target=validate-tsdoc

- Add Biome v2.3.13 configuration with custom rules
- Configure lint, format, and format:check targets in nx.json
- Add validate-tsdoc executor for documentation validation
- Update lefthook.yml with Biome pre-commit hooks
- Update validate-pr.yml workflow for Biome
- Update generators to include Biome configuration

This PR establishes the foundation for Biome-based linting and formatting
that all subsequent PRs will use.
- Remove opencode-font override from biome.json to enable linting/formatting
- Fix ASCIISiteTitle variable name typo (_siteTitle -> siteTitle)
- Fix non-null assertion in load-all-agent-specs.ts using type guard pattern

All checks now pass:
- Biome linting ✓
- Biome formatting ✓
- docs-builder build ✓
Use proper type predicate pattern instead of type assertion
for filtering successful agent spec loads.

Fixes CI typecheck failure.
- Fix type predicate in load-all-agent-specs.ts using proper pattern
- Fix 2 lint errors in validate-docs-links.ts (noImplicitAnyLet, useIterableCallbackReturn)
- Add missing imports to Header.astro (ASCIISiteTitle, SocialIcons, Search, ThemeSelect)

Typecheck and lint now pass locally.
Apply Biome formatting to 4 package.json files:
- packages/opencode-agent-loader-plugin/package.json
- packages/opencode-agent-loader-plugin/tsconfig.test.json
- packages/opencode-font/package.json
- packages/opencode-warcraft-notifications-plugin/package.json

Fixes format CI failures on PR #21.
@github-actions
Copy link

github-actions bot commented Feb 1, 2026

⚠️ Large PR Warning

This PR is quite large with:

  • 24 changed files
  • 484 added lines
  • 226 deleted lines

Consider breaking it down into smaller, focused PRs for easier review.

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.

2 participants