Skip to content

fix(milestones): remove released issues from the allocations map - #637

Open
yggdrasilworld wants to merge 2 commits into
MergeFi:mainfrom
yggdrasilworld:fix/allocations-drift-608
Open

yggdrasilworld wants to merge 2 commits into
MergeFi:mainfrom
yggdrasilworld:fix/allocations-drift-608

Conversation

@yggdrasilworld

@yggdrasilworld yggdrasilworld commented Oct 14, 2026 •

Copy link
Copy Markdown

What — phantom ledger drift

release_issue marked the issue Released + paid out, but left the entry in milestone.allocations — the map grew into a phantom ledger where 'allocated' no longer meant outstanding.

Fix

Remove the entry on release. (remaining_budget was already reduced at allocate-time, so it's correctly untouched.)

Resolves #608

Payment: Base USDC 0xD40Df5c27cEa7dd78A3c345401995c319589334e

release_issue marked the issue Released and paid out, but left the entry in
milestone.allocations — the map grew into a phantom ledger where 'allocated'
no longer meant outstanding. Remove it on release (remaining_budget was
already reduced at allocate-time, so it's untouched here).

Resolves MergeFi#608
@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@yggdrasilworld is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

@yggdrasilworld

Copy link
Copy Markdown
Author

Verification note

The touched Rust sources parse cleanly — balanced structure, no duplicate functions, no merge artifacts. cargo test couldn't run locally: the repo pins toolchain 1.82.0 + wasm32v1-none (unavailable for that channel) and the crates declare rust-version = 1.95.0; my sandbox's linker is also blocked pending an Xcode license. Diff-level verification: release_issue removes the released entry from milestone.allocations. The contracts/*/test.rs additions are included in the diff; the suite will exercise them under the repo's CI.

…es allocation removal; rewire orphaned test imports

This branch has not been deployed

No deployments
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.

allocations map retains released issues — phantom ledger drift

1 participant