Skip to content

fix(ci): resolve Format Check failures and guard Dependabot major bumps#52

Merged
RISHII7 merged 3 commits into
mainfrom
develop
Jul 9, 2026
Merged

fix(ci): resolve Format Check failures and guard Dependabot major bumps#52
RISHII7 merged 3 commits into
mainfrom
develop

Conversation

@RISHII7

@RISHII7 RISHII7 commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

Fast-tracks two CI-only fixes from develop to main, both currently broken on main since the v1.0.0 merge:

  • Format Check was failing on apps/embed/package.json (missing trailing newline) and apps/widget/public/widget.js (pre-built bundle, now excluded from Prettier like it already is from ESLint).
  • Dependabot now ignores semver-major updates for apps/web, apps/widget, and packages/ui (previously only the root workspace had this guard), preventing runtime-mismatched bumps like @types/node 20→26 against the Node 22.15.1 pinned in every workflow.

No application code changes — CI config and generated-file exclusions only.

Summary by CodeRabbit

  • Chores
    • Adjusted update settings to skip major dependency upgrades in several app packages.
    • Removed reviewer assignment from automated update entries.
    • Expanded formatting ignore rules for generated and documentation files.
    • Applied a minor formatting cleanup in one app package file.

RISHII7 added 3 commits July 9, 2026 10:13
The root workspace entry already ignored semver-major updates, but the
apps/web, apps/widget, and packages/ui entries did not, letting through
PRs like the @types/node 20->26 bump that broke CI (Node runtime is
pinned to 22.15.1 everywhere) and left pnpm-lock.yaml out of sync.

Also drops the `reviewers` key from every entry: GitHub deprecated it
in favor of CODEOWERS-based review requests, so it was already a no-op.
- apps/embed/package.json was missing a trailing newline
- apps/widget/public/widget.js (the pre-built, minified embed loader
  bundle) is now excluded from Prettier, matching its existing ESLint
  exclusion — it is shipped output, not source

These two files have been failing the Format Check job on develop
and main since the v1.0.0 merge.
ci: guard all Dependabot ecosystems against major-version bumps
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f63ab64b-7a55-4bb7-832a-e70b60883797

📥 Commits

Reviewing files that changed from the base of the PR and between c7f219b and 6244f87.

📒 Files selected for processing (3)
  • .github/dependabot.yml
  • .prettierignore
  • apps/embed/package.json

📝 Walkthrough

Walkthrough

This PR updates repository configuration files: dependabot.yml adds ignore rules for major version updates across npm scopes and removes reviewer assignments from all update entries, .prettierignore adds entries for generated/pre-built files, and apps/embed/package.json receives a minor formatting adjustment.

Changes

Config and Tooling Maintenance

Layer / File(s) Summary
Dependabot ignore rules and reviewer removal
.github/dependabot.yml
Adds ignore rules skipping semver-major updates for all dependencies across root, web, widget, and UI npm scopes, and removes the reviewers field from all npm and GitHub Actions update entries.
Prettierignore and package.json formatting
.prettierignore, apps/embed/package.json
Adds ignore entries for packages/backend/convex/README.md and the pre-built apps/widget/public/widget.js bundle, and applies a minor end-of-file formatting adjustment to embed's package.json.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

  • RISHII7/echo#20: Modifies the CI formatting step to use --ignore-path .prettierignore, relying on the same ignore file content updated in this PR.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the CI formatting fix and the Dependabot major-update guard added in this PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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