Skip to content

Fix forge-std remapping for invariant CI#62

Merged
abhicris merged 1 commit into
kcolbchain:mainfrom
bonginkan:codex/fix-forge-std-remapping-20260603
Jun 4, 2026
Merged

Fix forge-std remapping for invariant CI#62
abhicris merged 1 commit into
kcolbchain:mainfrom
bonginkan:codex/fix-forge-std-remapping-20260603

Conversation

@MyTH-zyxeon
Copy link
Copy Markdown
Contributor

Summary

  • fixes the Forge remapping introduced with the invariant tests
  • points forge-std/ imports at the installed Foundry library under lib/forge-std/src/
  • keeps the change scoped to foundry.toml

Why

main and #45 currently fail forge test because forge-std/StdInvariant.sol resolves through the local forge-std/ shim directory, which only contains Test.sol. The workflow already installs foundry-rs/forge-std, so the remapping should target that installed package instead.

Validation

  • GitHub Actions should rerun the existing forge workflow on this one-line config change.

@abhicris
Copy link
Copy Markdown
Contributor

abhicris commented Jun 4, 2026

🤖 Audit verdict: safe

Routine Foundry configuration fix updating a circular/broken remapping to the standard library path; no code execution, secrets, or supply-chain risk.

Audited by the kcolbchain PR pipeline. See pipeline docs.

@abhicris abhicris merged commit 73fc3dc into kcolbchain:main Jun 4, 2026
1 check passed
@abhicris
Copy link
Copy Markdown
Contributor

abhicris commented Jun 4, 2026

Merged — thanks, @MyTH-zyxeon. That's 9 merged PRs to kcolbchain now.

You're a Fellow — when paid research, partner-org work, or grant milestones come up, you're in the first invite pool.

Next-up issues across the org: https://kcolbchain.com/invitations/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants