test: reduce redundant SIDC coverage

[the-Drunken-coder]

Summary

• remove redundant expanded catalog render/build tests now covered by the catalog-wide round-trip test
• drop unused SIDC constants tied to those duplicate cases

Validation

• npm test
• npm pack --dry-run
• git diff --check HEAD~1..HEAD

Summary by CodeRabbit

• Tests
  • Simplified the SIDC test suite by removing a large set of expanded fixture cases.
  • Kept coverage for core curated symbol handling and broader public API round-trip checks.
  • Streamlined test maintenance without changing end-user behavior.

Greptile Summary

This PR reduces duplicate SIDC test coverage in the test suite. The main changes are:

• Removed redundant expanded catalog render checks.
• Removed duplicate expanded catalog buildSidc fixture assertions.
• Dropped unused SIDC constants tied to the removed tests.

Confidence Score: 5/5

Safe to merge; the changes only remove redundant test cases and unused constants from the test suite.

The diff is limited to test coverage cleanup in a single file, with no production code changes or behavioral changes.

T-Rex Logs

What T-Rex did

• Ran the pre-run tests for reduce-sidc-coverage and confirmed npm test finished with all tests passing, including base expandedCatalogSidcs and explicit expanded render/build SIDC tests/constants.
• Reviewed the post-run state, where the PR diff deletes the duplicate test/constants, preserves all curated catalog entries round-trip through public APIs, and confirmed npm test, npm pack --dry-run, and git diff --check HEAD~1..HEAD exit code 0.

_{Ran code and verified through T-Rex}

_{Reviews (1): Last reviewed commit: "test: reduce redundant SIDC coverage" | Re-trigger Greptile}

[coderabbitai]

📝 Walkthrough

Walkthrough

The SIDC test file trims its curated fixture constants and removes expanded-curated SIDC assertion coverage, while keeping the broader symbol-identification and search tests intact.

Changes

SIDC test fixture cleanup

Layer / File(s)	Summary
Curated fixture pruning and coverage removal test/sidc-kit.test.mjs	The shared SIDC fixture block keeps a smaller curated set, and the expanded-curated renderSymbol and buildSidc assertions are removed from the test suite.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• the-Drunken-coder/sidc-kit#1: Introduces the SIDC test suite and fixture setup that this PR trims.
• the-Drunken-coder/sidc-kit#4: Adds renderSymbol coverage for SIDC cases in the same test file that this PR removes for expanded curated fixtures.

Poem

🐰 I nibbled the fixture carrots neat,
and left four SIDCs in a row.
The extra hops went off to rest,
while symbols still know how to glow.
Hop hop, the test burrow stays tidy!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: reducing redundant SIDC test coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/reduce-redundant-sidc-tests

---

_{Comment @coderabbitai help to get the list of available commands.}