Skip to content

quest: plan fixes for main's red ctgrind and build nightlies - #591

Merged
kixelated merged 2 commits into
mainfrom
quest-plan/nightly-fixes
Nov 11, 2026
Merged

kixelated merged 2 commits into
mainfrom
quest-plan/nightly-fixes

Conversation

@kixelated

@kixelated kixelated commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

Plans the fixes for the nightly failures that moq-dev/rind#575's PR runs surfaced on main. A triage of the logs found six causes. None is a constant-time violation in shipped code.

Changes

  • New quest/m0/ctgrind-nightly.md [S]:
    • cipher::ChaCha20::new and ed25519's seed path mark the caller's key buffer secret, which poisons a later test that reads the same constant.
    • The guard-page over-read test can't fault under valgrind on ARM.
    • The rind-rcgen PEM suppressions lack armv7's Value4 form.
  • New quest/m0/build-nightly.md [XS]:
    • A Dart hook test times out on the Intel Mac runner.
    • qemu-user-static is gone on ubuntu 26.04.
    • moq-dev/rind#581's leak test is optimized away on the qemu targets.

Decisions

Settled by the maintainer on 2026-11-10 (/quest-complete, /quest-plan):

  • ✅ Two m0 quests, one per workflow (rejected: one quest for all six causes; drop).
  • Guard-page test under ctgrind: ✅ leave the guard_pages module out of __ctgrind builds (rejected: skip it only on ARM; detect valgrind at run time).

Models

Size: XS
Implementer: Claude Code, 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: 01a1c7ec-dfd5-7000-8475-7eb4d2158c20, round 1
  • Head: 4e9a66b
  • Base: origin/main at cadc077
  • Score: 10/10, The change adds only two quest planning documents, changes no executable code or tests, and has no actionable defects.
  • Findings: none

kixelated and others added 2 commits November 10, 2026 16:04
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated
kixelated marked this pull request as ready for review November 11, 2026 00:06
@pullfrog

pullfrog Bot commented Nov 11, 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 balance. Top up at https://opencode.ai/console/wrk_01M28N93AE612R8VKHHW47PJER/billing?credits=1. (402)

Top up OpenCode Go →

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

@coderabbitai

coderabbitai Bot commented Nov 11, 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 39 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: e8aac2a5-ee10-42f4-9c31-11b162d1799d
Commits

Reviewing files that changed from the base of the PR and between cadc077 and 4e9a66b.

Files selected for processing: 2

  • quest/m0/build-nightly.md
  • quest/m0/ctgrind-nightly.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

@kixelated
kixelated added this pull request to the merge queue Nov 11, 2026
Merged via the queue into main with commit e7c3327 Nov 11, 2026
4 of 5 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