feat(verify-bytecode): ignore immutable refs during runtime check

[yash-atreya]

By skipping immutables, do you mean we don't check the immutables that are embedded in the runtime code?
If yes, then NO; we do check them. Runtime codes are compared entirely.

Got it, in general I think that's the better and stronger approach. To take a step further, you can do: if no constructor arguments are passed AND no API key is passed (so forge has no constructor args), then it skips checking immutables. That would help cases like #8617. In this case you would also want to include a message in the output indicating this difference

Originally posted by @mds1 in #8510 (comment)

[klkvr]

we could reuse

foundry/crates/common/src/contracts.rs

Line 148 in c6d342d

pub fn find_by_deployed_code_exact(&self, code: &[u8]) -> Option<ArtifactWithContractRef<'_>> {

for this