Skip to content

Update demo style defaults#879

Open
mk3008 wants to merge 1 commit into
mainfrom
codex/demo-style-defaults
Open

Update demo style defaults#879
mk3008 wants to merge 1 commit into
mainfrom
codex/demo-style-defaults

Conversation

@mk3008

@mk3008 mk3008 commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Update the demo PostgreSQL style preset to use the requested formatting defaults.
  • Align MySQL and SQL Server demo presets to the same layout defaults while keeping DBMS-specific identifier escape and parameter settings.
  • Align CUD and migration demo fallback formatter defaults with the shared demo layout defaults.

Verification

  • node --check docs\public\demo\style-config.js
  • node --check docs\public\cud-demo\script.js
  • node --check docs\public\migration-demo\script.js
  • node --input-type=module -e "... style-defaults smoke ..."
  • pnpm --filter rawsql-ts test -- SqlFormatter.join-condition-layout.test.ts SqlFormatter.oneline-max-length.test.ts SqlFormatter.identifier-minimal.test.ts
  • pnpm test
  • Pre-commit hook: staged-file policy, pnpm typecheck, pnpm test, pnpm build, and pnpm lint

Merge Readiness

  • No baseline exception requested.
  • Baseline exception requested and linked below.

Tracking issue:
Scoped checks run:
Why full baseline is not required:

Self Review

Self-review workflow: developer-self-review skill, consistency review, and human acceptance review.
Self-review result: No blockers remain; the diff is limited to demo preset and fallback default configuration.
Concept-review workflow: Explicit no-concept-impact review; this PR changes browser demo style defaults only.
Concept-review result: No package behavior, scaffold contract, or concept-boundary violation found.

CLI Surface Migration

  • No migration packet required for this CLI change.
  • CLI/user-facing surface change and migration packet completed.

No-migration rationale: This is a browser demo preset/default update, not a CLI behavior or option migration.
Upgrade note: No upgrade action required.
Deprecation/removal plan or issue: No deprecation or removal.
Docs/help/examples updated: Demo presets and fallback defaults were updated.
Release/changeset wording: No package release changeset; docs/demo only.

Scaffold Contract Proof

  • No scaffold contract proof required for this PR.
  • Scaffold contract proof completed.

No-proof rationale: This PR does not change scaffold generation, scaffold inputs, or generated scaffold output.
Non-edit assertion: Not applicable; no scaffold files are edited.
Fail-fast input-contract proof: Not applicable; no scaffold input contract changes.
Generated-output viability proof: Not applicable; no scaffold output changes.

@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@mk3008, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 34 minutes and 56 seconds. Learn how PR review limits work.

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c24744fc-0be5-48f1-bd81-30aa4e750681

📥 Commits

Reviewing files that changed from the base of the PR and between dae1ef7 and 1526f8f.

📒 Files selected for processing (3)
  • docs/public/cud-demo/script.js
  • docs/public/demo/style-config.js
  • docs/public/migration-demo/script.js
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/demo-style-defaults

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 and usage tips.

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