Skip to content

tasks: doc comment back on retry_unstored_hands - #903

Merged
svanlabs[bot] merged 2 commits into
mainfrom
fix/hands-doc-comment
Oct 10, 2026
Merged

svanlabs[bot] merged 2 commits into
mainfrom
fix/hands-doc-comment

Conversation

@svanlabs

@svanlabs svanlabs Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Follow-up to #897 (PR #901/#902). The doc block for retry_unstored_hands had ended up on flush_unstored_hands. Doc-only move; scripts/check.sh full green.

Generated-by: claude-code/claude-haiku-5-5

🤖 Generated with Claude Code

Validation refreshed at 3bf0077f64787ea9035ff141267750adb2ce2bd8 against current main: shared local review runner/full gate passed, 892 passed, 0 failed, 4 ignored, 35 binaries. Independent Standards and Spec reviews found no substantive issues.

Generated-by: codex/gpt-6

The flush_unstored_hands change (#897) put its doc comment between the
doc block of retry_unstored_hands and that fn, so the bounded-retry
note now documented the wrong function. Move it back.

Generated-by: claude-code/claude-haiku-5-5

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@svanlabs
svanlabs Bot force-pushed the fix/hands-doc-comment branch from 98910bd to 8cdd55b Compare October 7, 2026 21:03
Validate pull request 903 against the current released source.

Generated-by: codex/gpt-6

@svanlabs svanlabs Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed integration commit 3bf0077f64787ea9035ff141267750adb2ce2bd8 against 15c3d730814f5e9465b6dc0b34073052b071fee3 using the shared local review runner and independent Standards and Spec reviewers.

Standards

No substantive violations found. The existing four-line documentation block now immediately precedes retry_unstored_hands, and exit-flush documentation remains on flush_unstored_hands. This is a focused documentation correction with no executable, architecture, strategy or performance change.

Spec

No findings. The move matches the follow-up to #897: the retry documentation accurately describes the unchanged tuple return, watermark handling and bounded requeue behavior. No scope expansion.

Validation: exact-commit full gate passed: 892 tests passed, 0 failed, 4 ignored; 35 test binaries. No behavior or performance change requiring simulation or measurement.

Summary: Standards 0 findings; Spec 0 findings.

Generated-by: codex/gpt-6

@svanlabs
svanlabs Bot enabled auto-merge October 10, 2026 06:44
@svanlabs
svanlabs Bot merged commit 5c8d0e7 into main Oct 10, 2026
3 of 6 checks passed
@svanlabs
svanlabs Bot deleted the fix/hands-doc-comment branch October 10, 2026 06:50
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.

0 participants