Skip to content

chore(quest): HKDF-SHA256 expand slowed on desktop; confirm, then bisect - #485

Merged
kixelated merged 1 commit into
mainfrom
quest-plan/hkdf-expand-regression
Nov 7, 2026
Merged

kixelated merged 1 commit into
mainfrom
quest-plan/hkdf-expand-regression

Conversation

@kixelated

@kixelated kixelated commented Nov 7, 2026 •

Copy link
Copy Markdown
Collaborator

Plans a follow-up from moq-dev/rind#484: its desktop (Arrow Lake) run measured hkdf-sha256/expand/rind at 66 ns at 9e7f6a5a, against 47 ns at f95b91da in the previous run. rind is still the fastest library on that row, and kixtop did not move.

Adds quest/m2/speed/hkdf-expand-regression.md [XS] to the m2 speed epic.

Decisions

  • Confirm on a GitHub x86_64 bench runner before fixing, then bisect the 23 rind commits between the two runs if it reproduces (maintainer, 2026-11-07).
  • Milestone m2/speed, since it opens no gap in benchmarks/gaps.toml (maintainer, 2026-11-07).
  • The plan names moq-dev/rind#406 as the first suspect but notes that only PBKDF2 calls the tag it put out of line, and lists moq-dev/rind#416 and moq-dev/rind#412 as likelier.

Models

Size: XS
Implementer: Claude Code, Claude Opus 5.5, unknown effort

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

Review

Rewritten by quest review after each round.

  • Reviewer: Codex, gpt-6.1-sol, low effort
  • Session: 01a1b69d-ab13-7e42-8a56-6e5ebc3e7d78, round 1
  • Head: 41f7fae
  • Base: origin/main at 6406272
  • Score: 10/10, The specified diff only adds an HKDF performance-investigation quest and its index link; it introduces no executable changes or actionable defects under the repository’s review rules.
  • Findings: none

…er, then bisect

Follow-up from #484.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated
kixelated marked this pull request as ready for review November 7, 2026 15:26
@pullfrog

pullfrog Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

Your OpenCode Go account is out of credit. Top it up, then re-trigger Pullfrog.

OpenCode Go's response
provider error: Insufficient account funds

Top up OpenCode Go →

Pullfrog  | Rerun failed job ➔ | View workflow run | via Pullfrog | Using glm-5.3

@kixelated
kixelated added this pull request to the merge queue Nov 7, 2026
@coderabbitai

coderabbitai Bot commented Nov 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

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

Next included review available in 40 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1b3f203f-08a4-4237-9794-fbea41e136e8
Commits

Reviewing files that changed from the base of the PR and between 6406272 and 41f7fae.

Files selected for processing: 2

  • quest/m2/speed/README.md
  • quest/m2/speed/hkdf-expand-regression.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Merged via the queue into main with commit 165545e Nov 7, 2026
7 of 8 checks passed
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