Skip to content

chore: add repo refresh onboarding and CI - #42

Merged
edhor1608 merged 2 commits into
mainfrom
repo-refresh-onboarding-ci
May 2, 2026
Merged

edhor1608 merged 2 commits into
mainfrom
repo-refresh-onboarding-ci

Conversation

@edhor1608

@edhor1608 edhor1608 commented May 1, 2026 •

Copy link
Copy Markdown
Owner

Add CI workflow, Biome linting, and doctor preflight script for repo onboarding

  • Adds a GitHub Actions CI workflow that runs typecheck, lint, doctor, and fast tests on PRs and pushes to main using Bun 1.3.6 and ffmpeg.
  • Introduces src/cli/doctor.ts to validate the local toolchain (Bun, ffmpeg, ffprobe), fixture presence, and writable directories, exiting non-zero on failure.
  • Adds biome.json with recommended lint rules and noExplicitAny as an error; updates package.json with lint, typecheck, check, test:fast, and test:ci scripts with explicit 30s timeouts.
  • Fixes loadRuleset in src/domain/rules.ts to resolve the ruleset path relative to the module file rather than process.cwd(), and adds validation for required top-level keys.
  • Fixes PPM parsing in tests/helpers/image.ts to skip comment lines and strictly validate pixel count; diffRgb.meanChannelDelta now reports per-pixel delta (3× larger than before).
  • Behavioral Change: loadRuleset now throws if profiles or white_canvas keys are missing, and resolves config from the module path — callers relying on CWD-relative resolution will break.

Macroscope summarized 2e64b0d.

@coderabbitai

coderabbitai Bot commented May 1, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@edhor1608 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 6 minutes and 40 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7990cb25-0db5-4f5c-bc3a-114b8c722365

📥 Commits

Reviewing files that changed from the base of the PR and between c019724 and 2e64b0d.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • .github/workflows/ci.yml
  • AGENTS.md
  • README.md
  • biome.json
  • docs/repo_refresh_audit.md
  • package.json
  • src/cli/doctor.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch repo-refresh-onboarding-ci
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch repo-refresh-onboarding-ci

Review rate limit: 0/5 reviews remaining, refill in 6 minutes and 40 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2e64b0d6ff

ℹ️ 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".

Comment thread src/cli/doctor.ts

function runChecks(): Check[] {
return [
{ name: "bun", ok: Bun.version.length > 0, detail: Bun.version },

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Enforce minimum Bun version in doctor checks

The new preflight reports Bun as healthy whenever Bun.version is non-empty, so unsupported versions (for example Bun 1.2.x) still pass even though the repo prerequisites require Bun 1.3+. In that scenario bun run doctor gives a false green signal and users can proceed into later failures caused by version mismatch; the check should parse and compare the semantic version against the minimum supported version.

Useful? React with 👍 / 👎.

@edhor1608
edhor1608 changed the base branch from repo-refresh-config-hardening to main May 2, 2026 21:17
@edhor1608
edhor1608 merged commit c754e7f into main May 2, 2026
3 checks passed
@edhor1608
edhor1608 deleted the repo-refresh-onboarding-ci branch May 15, 2026 18:54
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