build: add CI workflow for format check and markdown audit#635
build: add CI workflow for format check and markdown audit#635saif-at-scalekit wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
🧰 Additional context used🧠 Learnings (2)📓 Common learnings📚 Learning: 2026-04-20T11:29:40.242ZApplied to files:
🔇 Additional comments (2)
WalkthroughA new GitHub Actions CI workflow is introduced that executes on all branch pushes and pull requests. The workflow sets up Node.js 20 with pnpm, installs dependencies using a frozen lockfile, then runs Astro type checking, format validation, and a strict markdown audit script. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
✅ Deploy Preview for scalekit-starlight ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Core: Astro 6 upgrade
- Upgraded from Astro 5 → 6 with all compatible starlight plugins
- Fixed Astro 6 build errors: image imports changed from string paths to imported assets,
3 anchor IDs changed due to GitHub-compatible slug generation
- Recreated vite-plugin-static-copy patch for v4.1.0
Package.json cleanup
- Removed husky (project uses simple-git-hooks)
- Moved pnpm, @types/react, @types/react-dom to devDependencies
- Added packageManager: pnpm@10.33.1 field; removed redundant pnpm from devDependencies
- Added simple-git-hooks to pnpm.onlyBuiltDependencies
- Patch-level dep bumps across several packages
AgentKit content
- Restructured bring-your-own-connector section; renamed pages to match titles
- Added MCP provider support to BYOC docs
- Fixed executeTool examples with correct connection names and account IDs
- Added HeyReach connector docs
LLM/AI discovery
- Improved generate-llms-index.js: AgentKit section grouping, framework description
accuracy, withFileTypes optimization, dead code removed
- Fixed framework routing URLs in llms-judge-prompt.txt (/frameworks/ → /examples/,
removed Agno/GenAI/MCP entries that have no pages)
- Added LLM judge prompt for comparing llms.txt versions
Infrastructure
- Added OAuth/OIDC discovery redirects (/.well-known/openid-configuration,
/.well-known/oauth-authorization-server) to redirects.config.ts
- Removed preview URL logging from pre-push hook (Netlify Pro billing change)
- Moved CI workflow to separate PR #635
CodeRabbit fixes
- Fixed curl single-quote bug in create-connector.mdx (env vars weren't expanding)
- Fixed trailing-hyphen anchor links in add-modular-sso.mdx and sso/quickstart.mdx
- Fixed stale BYOC page links in llms-judge-prompt.txt
|
Closing this PR as it falls short of the intended objective. The CI pipeline should trigger a full development build and surface any resulting errors, not just run local checks. |
Summary
.github/workflows/ci.ymlto run on every push and PRpnpm run format:check) and agent markdown audit (node scripts/agent-markdown-audit.js)Summary by CodeRabbit