[lint-monster] LintMonster: remove shared mutable package state in pkg/parser/virtual_fs.go #53012

Summary

Daily make golint-custom found 1 custom-linter finding for shared mutable package state.

Finding

• pkg/parser/virtual_fs.go: package-level slice/map variable builtinVirtualFiles is mutated via index assignment; mutating shared package state risks data races and can leak state across calls

Root cause

A package-level collection is being modified after initialization. That introduces shared mutable state, which makes behavior order-dependent and can create concurrency hazards.

Expected outcome

Refactor the pkg/parser/virtual_fs.go code path so it no longer mutates shared package state while preserving behavior and keeping edits minimal.

Remediation checklist

• Keep remediation scoped to the builtinVirtualFiles mutation finding only
• Prefer minimal, targeted code edits; clone or reconstruct data locally instead of mutating package-global state
• Preserve behavior and existing tests; widen scope only if required to eliminate shared mutation cleanly
• Validate the final result with make golint-custom

Copilot execution guidance

Use skill-fused constraints for this lint group:

1. Work only on the shared mutable package-state finding in pkg/parser/virtual_fs.go.
2. Prefer a local copy, immutable data flow, or helper extraction over broader refactors.
3. Preserve existing behavior and keep public surface changes to the minimum necessary.
4. Run make golint-custom before finishing and ensure this finding is resolved.

Generated by 🧌 LintMonster · gpt54 · 13.3 AIC · ⌖ 5.42 AIC · ⊞ 5.9K · ◷

• expires on Aug 22, 2026, 6:52 PM UTC-08:00