fix(core): use globbing also for workspace file patterns without '*'

[miluoshi]

This change allows cache input strings for{workspaceRoot} filesets to be parsed as glob patterns even if they don't contain asterisk * inside.

Few examples:

// nx.json 
{
  "targetDefaults": {
    "build": {
      "inputs": [
        "{workspaceRoot}/.babelrc?(.json)", 
        "{workspaceRoot}/tsconfig.{e2e-base,ts-node}.json"
      ]
    }
}
// or project.json
{
  "targets": {
    "build": {
      "inputs": [
        "{workspaceRoot}/.babelrc?(.json)", 
        "{workspaceRoot}/tsconfig.{e2e-base,ts-node}.json"
      ]
    }
  }
}

Making perf optimizations by skipping minimatch for a few static files might not be worth the tradeoff. (related commit: 15ccae0)

Note: this is a behaviour related to "cache inputs" feature introduced in Nx 14.4.

Current Behavior

Rerunning a target with above defined inputs with one of .babelrc or tsconfig.{e2e-base,ts-node}.json files updated will not invalidate a cache for the target.

Expected Behavior

Those files would get matched by Hasher and the Nx cache for the target would get invalidated.

[nx-cloud-staging]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ac15309. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 12 targets

• nx affected --target=e2e --base=f2222ff9689f48162e84d0e393c71384f640cd04 --head=ac15309aaaea6134c47a6a0e9c15bc598b084480 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected --target=test --base=f2222ff9689f48162e84d0e393c71384f640cd04 --head=ac15309aaaea6134c47a6a0e9c15bc598b084480 --parallel=1
• nx affected --target=build --base=f2222ff9689f48162e84d0e393c71384f640cd04 --head=ac15309aaaea6134c47a6a0e9c15bc598b084480 --parallel=3
• nx affected --target=lint --base=f2222ff9689f48162e84d0e393c71384f640cd04 --head=ac15309aaaea6134c47a6a0e9c15bc598b084480 --parallel=3
• nx-cloud record -- yarn nx workspace-lint
• nx-cloud record -- yarn documentation
• nx build typedoc-theme
• nx-cloud record -- yarn depcheck
• nx-cloud record -- yarn check-lock-files
• nx-cloud record -- yarn check-commit
• nx-cloud record -- yarn nx format:check --base=f2222ff9689f48162e84d0e393c71384f640cd04 --head=ac15309aaaea6134c47a6a0e9c15bc598b084480
• nx-cloud record -- yarn check-imports

---

Sent with 💌 from NxCloud.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Jul 26, 2022 at 6:19PM (UTC)

[miluoshi]

Alternatively, condition checking for presence of minimatch pattern (fileset.indexOf('*') > -1) could be expanded to multiple most common glob patterns, e.g. /[*?]|{.+}|[+@!]\(/.test(fileset) (matching patterns: *, ?, ?(), +(), @(), !(), {a,b})

[FrozenPandaz]

Thank you for the contribution!

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.