BREAKING: Make same-file duplicate skipping the default

[Nic-Polumeyv]

Reworked per the discussion in #601: content-hash matching is removed and skipDuplicates stays a boolean. no-duplicate.expected.css had the content behavior baked into its default expectations and is regenerated.

Fixes #601

[RyanZim]

@Nic-Polumeyv You'll want to rebase this now that #605 is merged. Also, are you up to modifying this to remove the content-hash-matching logic outright (keeping skipDuplicates a boolean option) as discussed in #601?

[Copilot]

Pull request overview

Changes duplicate skipping to deduplicate by resolved file path rather than identical content, preventing path-dependent semantics from being dropped.

Changes:

• Removes content-hash deduplication.
• Adds coverage for identical content at different paths.
• Updates expected output and option documentation.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
index.js	Removes hash-tracking state.
lib/parse-styles.js	Removes content-based duplicate checks.
README.md	Updates skipDuplicates documentation.
test/import.js	Registers the new regression fixture.
test/fixtures/same-file-dedup.css	Imports distinct and repeated paths.
test/fixtures/same-file-dedup.expected.css	Verifies path-based deduplication.
test/fixtures/imports/same-file-dedup/a/shared.css	Adds first identical source file.
test/fixtures/imports/same-file-dedup/b/shared.css	Adds second identical source file.
test/fixtures/no-duplicate.expected.css	Updates expectations without content deduplication.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[romainmenke]

Thank you. LGTM 👍

[RyanZim]

Thanks a lot @Nic-Polumeyv!