feat(forge): implement glob pattern for forge build --skip

[0xalpharush]

Motivation

Currently, forge build --skip test --skip script is unreliable as some projects do not consistently use .t.sol or .s.sol. This PR enables glob patterns like --skip */test/** --skip */script/** to enable only building src.

Closes #4628

Solution

I moved an existing glob matcher util to common and used it in the the FileFilter impl

[0xalpharush]

Ideally to match /private/var/folders/test/Foo.sol (this is how it appears in the tests) one wouldn't need */test/** and could instead use ./test/*. I don't have insight into how paths are passed around in Foundry (are they always absolute?), but I could appreciate clarification on this point.

For example, this is the cause of the failing test:

        let file = Path::new("/private/var/folders/test/Foo.sol");
        assert!(!SkipBuildFilter::Custom("*/test/**".to_string()).is_match(file)); // passes 
        assert!(!SkipBuildFilter::Custom("./test/**".to_string()).is_match(file)); // fails

        let file = Path::new("script/Contract.sol");
        assert!(!SkipBuildFilter::Custom("*/script/**".to_string()).is_match(file)); // fails
        assert!(!SkipBuildFilter::Custom("./script/**".to_string()).is_match(file)); // passes

[mattsse]

have insight into how paths are passed around in Foundry (are they always absolute?)

they're relative to the configured root when dealing solc, outside of solc they always should be absolute, so foundry should always have absolute paths

[mattsse]

nice, tysm for this.

a few nits, tests look good though

[mattsse]

have insight into how paths are passed around in Foundry (are they always absolute?)

they're relative to the configured root when dealing solc, outside of solc they always should be absolute, so foundry should always have absolute paths

[mattsse]

we need to update docs here

[0xalpharush]

updated

[Evalir]

lgtm! thank you for this!

just a super small nit for consistency

[mattsse]

lgtm
tysm!