Skip to content

docs: enforce KoliBri spelling and ESM imports#8053

Merged
deleonio merged 1 commit intodevelopfrom
feature/update-agent.md-for-kolibri-naming-and-esm-import
Jul 17, 2025
Merged

docs: enforce KoliBri spelling and ESM imports#8053
deleonio merged 1 commit intodevelopfrom
feature/update-agent.md-for-kolibri-naming-and-esm-import

Conversation

@deleonio
Copy link
Copy Markdown
Contributor

Summary

  • document that KoliBri spelling must be used
  • mention ESM import style as preferred

Testing

  • pnpm lint (fails: pnpm not found)

https://chatgpt.com/codex/tasks/task_e_68788ea89168832b8330aa270fd39285

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

Hydration Benchmark Report (vs Baseline)

✅ Top 5 Improvements (Best Speedups)

Component Current Baseline Δ% Result
kol-table-stateless 349.4ms 437ms -20%
kol-form 29.6ms 35.6ms -16.9%
kol-progress 30.1ms 32.8ms -8.2%
kol-input-radio 52ms 56.5ms -8%
kol-input-email 57.7ms 62.5ms -7.7%

🔻 Flop 5 Regressions (Worst Slowdowns)

Component Current Baseline Δ% Result
kol-drawer 40.9ms 25.6ms +59.8% 🔻
kol-card 29.9ms 22.6ms +32.3% 🔻
kol-modal 23.8ms 20.5ms +16.1% 🔻
kol-input-number 63.4ms 56.1ms +13% 🔻
kol-badge 77.1ms 70.1ms +10%
📋 Show all results
Component Current Baseline Δ% Result
kol-abbr 20.3ms 19.8ms +2.5%
kol-accordion 130.7ms 131ms -0.2%
kol-alert 113.5ms 105.9ms +7.2%
kol-avatar 25.7ms 26.9ms -4.5%
kol-badge 77.1ms 70.1ms +10%
kol-breadcrumb 95.2ms 97.6ms -2.5%
kol-button 58.5ms 63ms -7.1%
kol-button-link 50.9ms 50.7ms +0.4%
kol-card 29.9ms 22.6ms +32.3% 🔻
kol-details 130.1ms 134.9ms -3.6%
kol-drawer 40.9ms 25.6ms +59.8% 🔻
kol-form 29.6ms 35.6ms -16.9%
kol-heading 22.1ms 21.5ms +2.8%
kol-icon 67.7ms 65.1ms +4%
kol-image 19.7ms 19.3ms +2.1%
kol-input-checkbox 141.5ms 141.3ms +0.1%
kol-input-color 82.7ms 81.9ms +1%
kol-input-date 68.6ms 70.3ms -2.4%
kol-input-email 57.7ms 62.5ms -7.7%
kol-input-file 101.9ms 93.7ms +8.8%
kol-input-number 63.4ms 56.1ms +13% 🔻
kol-input-password 53.2ms 51.6ms +3.1%
kol-input-radio 52ms 56.5ms -8%
kol-input-text 55.3ms 59.3ms -6.7%
kol-link 53.8ms 49.8ms +8%
kol-link-button 55.2ms 56.5ms -2.3%
kol-modal 23.8ms 20.5ms +16.1% 🔻
kol-nav 32.9ms 33.9ms -2.9%
kol-pagination 448.3ms 423.8ms +5.8%
kol-popover-button 71.5ms 71.3ms +0.3%
kol-progress 30.1ms 32.8ms -8.2%
kol-quote 19.3ms 19ms +1.6%
kol-select 120.4ms 117.1ms +2.8%
kol-skip-nav 23.8ms 23.6ms +0.8%
kol-spin 26.4ms 24.2ms +9.1%
kol-split-button 180.2ms 174.7ms +3.1%
kol-table-stateful 372.4ms 374.7ms -0.6%
kol-table-stateless 349.4ms 437ms -20%
kol-tabs 30.6ms 32.3ms -5.3%
kol-textarea 53.2ms 54.1ms -1.7%
kol-toolbar 35ms 36ms -2.8%
kol-tree 25.8ms 26.5ms -2.6%
kol-tree-item 66.2ms 64ms +3.4%
kol-version 152.5ms 154.8ms -1.5%

@deleonio deleonio requested a review from Copilot July 17, 2025 07:03
@deleonio deleonio removed the codex label Jul 17, 2025
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

This PR updates the style guidelines to enforce consistent KoliBri spelling and prefer ESM import syntax.

  • Add rule to use "KoliBri" casing everywhere (with an exception)
  • Encourage using ESM import statements instead of require
Comments suppressed due to low confidence (2)

AGENTS.md:233

  • [nitpick] Consider clarifying the exception for the KolKolibri component by specifying its file or usage context to avoid confusion for contributors.
- Spell "KoliBri" with this casing in all documentation and code. The only exception is the component named KolKolibri.

AGENTS.md:234

  • [nitpick] It may help to include a brief example of the preferred ESM import style and note any fallback pattern for environments where ESM isn't supported yet.
- Use ESM import syntax in browser code and scripts whenever supported, instead of `require` imports.

@deleonio deleonio marked this pull request as ready for review July 17, 2025 07:04
@deleonio deleonio requested a review from laske185 July 17, 2025 07:05
@deleonio deleonio merged commit 08b8d90 into develop Jul 17, 2025
10 checks passed
@deleonio deleonio deleted the feature/update-agent.md-for-kolibri-naming-and-esm-import branch July 17, 2025 07:36
@publicuibot publicuibot bot locked and limited conversation to collaborators Jul 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants