Skip to content

deps: ark-ui 5.38.1 -> 5.39.0 - #43

Draft
Guria wants to merge 1 commit into
mainfrom
agent/karkas-arkui-539
Draft

deps: ark-ui 5.38.1 -> 5.39.0#43
Guria wants to merge 1 commit into
mainfrom
agent/karkas-arkui-539

Conversation

@Guria

@Guria Guria commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Changelog-driven minor bump of @ark-ui/react 5.38.1 → 5.39.0 in apps/demo and the create-karkas template.

  • Pin bump only; lockfile via nub update @ark-ui/react (in-range transitive refresh: astro 7.2.4→7.2.7).
  • Full nub run test in apps/demo: 416/416 passed (26 files, storybook/chromium). A previous run 2 failures in Items.stories.tsx did not reproduce.
  • Pre-commit gate (format/lint/typecheck/fallow) green.

Follow-up to #41. [Hermi 🤖 — from hermi.exe.xyz]

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 50 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 076a1a4c-2250-495d-93ab-24dd27d75643

📥 Commits

Reviewing files that changed from the base of the PR and between 8dcd5e5 and 573eafa.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • apps/demo/package.json
  • packages/create-karkas/template/package.json

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.

@github-actions

Copy link
Copy Markdown
Contributor

Fallow audit report

No GitHub PR/MR findings.

Generated by fallow.

@Guria
Guria marked this pull request as draft August 27, 2026 20:45
@Guria

Guria commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

CI is red — upstream regression in @ark-ui/react 5.39.0, not a karkas bug. Converting to draft.

Repro (deterministic, 4/4 runs): run only src/app/integration/Items.stories.tsx with 5.39.0 — No Matching Items story fails at I.applyStockFilter('Out of Stock'):

  1. selectOption(combobox /Category/i, "Electronics") → OK, but the category positioner stays mounted and visible (multi-select does not close on select).
  2. selectOption(combobox /Stock/i, "Out of Stock") → the Stock trigger click is swallowed; testing-library sees only the category options (All/Electronics/Furniture/Clothing/Food) — no Stock options ever render.

Facts gathered:

  • DOM after the Stock trigger click contains 10 hidden-native options (from Select.HiddenSelect), but the Stock positioner never opens; error dump shows the category list still open.
  • Full-suite runs mask it (timing); single-file runs fail deterministically on 5.39.0, pass 47/47 on 5.38.1 (same code, only pin differs).
  • Instrumenting kahraman selectOption with Escape + trigger-click retry makes single-file runs pass — confirms stuck-open-positioner, not a handler/a11y change.
  • 5.39.0 changelog has no Select-behavior fix that explains it; Presence changes (onEnterComplete) touch Select — possible suspect.

Plan: hold ark-ui at 5.38.1 (main) until upstream fix; candidate follow-ups: (a) report to chakra-ui/ark with this repro, (b) robustness retry in kahraman selectOption (its own release). Keeping this PR as draft for visibility of the investigation.

[Hermi 🤖 — from hermi.exe.xyz]

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