Skip to content

chore: add builder.io rules#3799

Open
astandrik wants to merge 2 commits intomainfrom
astandrik.3798
Open

chore: add builder.io rules#3799
astandrik wants to merge 2 commits intomainfrom
astandrik.3798

Conversation

@astandrik
Copy link
Copy Markdown
Collaborator

@astandrik astandrik commented Apr 12, 2026

Closes #3798

CI Results

Test Status: ⚠️ FLAKY

📊 Full Report

Total Passed Failed Flaky Skipped
608 604 0 1 3

😟 No changes in tests. 😕

Bundle Size: ✅

Current: 63.40 MB | Main: 63.40 MB
Diff: 0.00 KB (0.00%)

✅ Bundle size unchanged.

ℹ️ CI Information
  • Test recordings for failed tests are available in the full report.
  • Bundle size is measured for the entire 'dist' directory.
  • 📊 indicates links to detailed reports.
  • 🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a .builderrules guidance file to steer Builder.io/AI-assisted code generation toward established patterns in ydb-embedded-ui, aligned with existing repo conventions and tooling.

Changes:

  • Introduces “Do/Don’t” rules for React/TS, routing, data fetching (RTK Query + window.api), i18n, styling, and tables.
  • Documents verified repo commands from package.json and safety/permission prompts for higher-impact operations.
  • Lists key project entry points and “good example” files to copy local patterns from.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Builder.io agent rules file to codify repository-specific conventions and stable anchors for AI-assisted changes, aligning Builder usage with existing YDB Embedded UI patterns.

Changes:

  • Introduces .builderrules with “Do / Don’t” guidance for React/TS, RTK Query, i18n, tables, and safety checks.
  • Documents verified repo commands to use for validation and CI-friendly workflows.
  • Provides a set of stable-anchor paths (AGENTS.md, app wiring, API, i18n utilities) to guide navigation.

- Do not use JSX fragment shorthand `<>`.
- Do not hardcode user-facing strings, colors, or empty placeholders.
- Do not fetch directly in components or bypass `window.api`.
- Do not import components through `index.ts` re-exports unless the local pattern explicitly requires it. Do not create new re-export-only `index.ts` files.
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.

chore: add builder.io rules

2 participants