docs: sweep stale Kamino-harvest-stub framing across registry#320
Merged
Conversation
|
Deployment failed with the following error: Learn More: https://vercel.com/alrimarleskovars-projects?upgradeToPro=build-rate-limit |
This was referenced May 15, 2026
alrimarleskovar
pushed a commit
that referenced
this pull request
May 15, 2026
Fix CI lint failure on PR #320 — cosmetic only (markdown table column widths re-aligned to prettier's expected format). No semantic change. https://claude.ai/code/session_01YapZy1Z5gzbV5EammBkSQm
The harvest path was shipped (real `redeem_reserve_collateral` CPI as redeem-all + redeposit-principal round-trip) and #233 closed, but several places in the doc set still framed it as "stub returning realized=0" or "post-audit". This sweep aligns the docs with the code: - MAINNET_READINESS.md 4.5 — flipped from 🟡⛔ to ✅ with the real implementation summary; dropped #233 from the 4.1 canary gate list; rewrote § 7 critical-path summary to surface the Agave 2.x toolchain (#319) as the live blocker and remove harvest from the to-do list. - docs/security/audit-readiness.md "pitch vs shipped" row — harvest is shipped, not staged. - docs/security/mev-front-running.md § 2.4 + § 3 summary table — harvest is live; sandwich vector is bounded (slippage guard + `PrincipalLoss` revert), Jito bundling demoted to operator concern. - docs/operations/mainnet-canary-plan.md — canary yield branch is unconditional (not gated on #233). - README.md — Yield Waterfall paragraph reflects both deposit and harvest paths shipped. No code changes. Pure doc-debt sweep so the auditor reads a register that matches the codebase on day one. https://claude.ai/code/session_01YapZy1Z5gzbV5EammBkSQm
Fix CI lint failure on PR #320 — cosmetic only (markdown table column widths re-aligned to prettier's expected format). No semantic change. https://claude.ai/code/session_01YapZy1Z5gzbV5EammBkSQm
alrimarleskovar
force-pushed
the
docs/post-233-harvest-sweep
branch
from
50d9c01 to
4a70bce
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Doc-debt sweep: aligns the public-facing registry (
MAINNET_READINESS.md,README.md, audit + MEV + canary docs) with the codebase after the Kamino harvest path landed and closed #233.The harvest implementation has been live for a while (real
redeem_reserve_collateralCPI as a redeem-all + redeposit-principal round-trip inprograms/roundfi-yield-kamino/src/lib.rs:272), but several places in the doc set still framed it as "stub returning realized=0" or "staged behind audit". An auditor opening the repo today reads a register that doesn't match the code — bad signal during external review.Changes
MAINNET_READINESS.md§ 4.5PrincipalLossguard, slippage guard inherited from PR #124)MAINNET_READINESS.md§ 4.1MAINNET_READINESS.md§ 1.5/reputacaoDemo Studio reflection)MAINNET_READINESS.md§ 7 critical pathdocs/security/audit-readiness.md§ Pitch vs shippeddocs/security/mev-front-running.md§ 2.4PrincipalLossrevert); Jito bundling demoted to operator concerndocs/security/mev-front-running.md§ 3 summary tableharvest_yield+deposit_idle_to_yieldrows updated from❌ (stub)→⚠️ Yes — boundedwith the actual mitigationsdocs/operations/mainnet-canary-plan.md§ smoke flowREADME.mdYield Waterfall paragraphWhat this PR is NOT
Test plan
grep -rn "harvest.*stub\|realized=0\|harvest.*post-audit\|deposit ≠ harvest"returns no remaining hits in tracked docsGenerated by Claude Code