Skip to content

fix: preserve tuple args in useFreshCallback#2048

Merged
imbhargav5 merged 1 commit into
mainfrom
codex/rooks-maintainer-20260628
Jul 11, 2026
Merged

fix: preserve tuple args in useFreshCallback#2048
imbhargav5 merged 1 commit into
mainfrom
codex/rooks-maintainer-20260628

Conversation

@imbhargav5

Copy link
Copy Markdown
Owner

Summary\n- widen useFreshCallback to preserve full parameter tuples\n- remove the remaining any cast from useDebounceFn\n- add a regression test for multi-argument forwarding\n\n## Verification\n- ./node_modules/.bin/vitest run src/tests/useFreshCallback.spec.ts src/tests/useDebounceFn.spec.ts\n- ./node_modules/.bin/tsc -p ./tsconfig.json --noEmit

@codesandbox

codesandbox Bot commented Jun 28, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel

vercel Bot commented Jun 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rooks Ready Ready Preview, Comment Jul 11, 2026 7:37pm

@imbhargav5

Copy link
Copy Markdown
Owner Author

@mergify queue

@imbhargav5
imbhargav5 force-pushed the codex/rooks-maintainer-20260628 branch from 43f7e41 to 81e78d8 Compare July 11, 2026 18:57
@imbhargav5

Copy link
Copy Markdown
Owner Author

@mergify queue

@imbhargav5
imbhargav5 force-pushed the codex/rooks-maintainer-20260628 branch from 81e78d8 to 2628729 Compare July 11, 2026 18:57
@unbound-computer-app

Copy link
Copy Markdown

Unbound Reviewer agent review is running for this PR.

@imbhargav5
imbhargav5 merged commit 28c7762 into main Jul 11, 2026
3 of 5 checks passed
@imbhargav5
imbhargav5 deleted the codex/rooks-maintainer-20260628 branch July 11, 2026 18:57
@mergify

mergify Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

queue

☑️ Command queue ignored because it is already running from a previous command.

@mergify

mergify Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Waiting for
  • -closed [📌 queue requirement]
  • check-success=Build (test)
  • check-success=Test (pnpm test)
All conditions
  • -closed [📌 queue requirement]
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue rule default]:
      • check-success=Build (test)
      • check-success=Test (pnpm test)
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = Configuration changed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant