perf: Implement terminal reuse for tags sharing the same content regex

[Copilot]

Each tag in build_cfg() was generating its own terminal rule (M_i), even when multiple tags shared identical regex patterns. This created redundant grammar rules, particularly problematic for the common `/(?s:.*)/" default pattern.

Changes

• Terminal naming: M_i (per-tag) → T_i (per-unique-pattern)
• Deduplication: Track unique patterns via unique_pattern_terminals dict
• Reuse: Tags with identical regex reference the same terminal
• Documentation: Updated docstring example to reflect new T_i terminal naming

Impact

10 tags with default pattern: 10 terminals → 1 terminal (90% reduction)

# Before
m_0[capture, suffix="<|/MASKED|>"]: M_0
m_1[capture, suffix="<|/MASKED|>"]: M_1  
M_0: /(?s:.*)/
M_1: /(?s:.*)/  # duplicate

# After
m_0[capture, suffix="<|/MASKED|>"]: T_0
m_1[capture, suffix="<|/MASKED|>"]: T_0  # reuse
T_0: /(?s:.*)/  # single definition

Testing

Updated test expectations and added comprehensive test case for terminal reuse with complex patterns:

• Multiple tags sharing the same date pattern (\d{4}-\d{2}-\d{2}) correctly reuse terminal T_0
• Multiple tags sharing the same time pattern (\d{2}:\d{2}:\d{2}) correctly reuse terminal T_1
• Verifies that 4 tags with 2 unique patterns create only 2 terminals (instead of 4)

All tests verify that the optimization maintains correctness while improving efficiency.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>perf: Implement terminal reuse for tags sharing the same content regex (e.g., /(?s:.*)/)</issue_title>
<issue_description>In build_cfg(): There may be many tags with "/(?s:.*)/" pattern, which can be inefficient. Implement terminal reuse for tags sharing the same content regex (e.g., /(?s:.*)/)

TODO Line:

GIMKit/src/gimkit/dsls.py

Line 98 in 3fa5e49

# TODO: There may be many tags with "/(?s:.*)/" pattern, which can be inefficient.

This is an optimized version of PR #167 </issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes perf: Implement terminal reuse for tags sharing the same content regex (e.g., /(?s:.*)/) #168

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Ki-Seki]

@copilot Add new test cases to reflect this change

[Copilot]

@copilot Add new test cases to reflect this change

Added 5 additional test cases in commit 1dcc04b:

1. test_build_cfg_single_tag_no_reuse - Edge case with single tag
2. test_build_cfg_all_unique_patterns - All tags with different patterns (no reuse)
3. test_build_cfg_many_tags_same_pattern - High reuse scenario with 10 tags (90% reduction)
4. test_build_cfg_complex_regex_patterns - Complex patterns with special characters (dates, times)
5. test_build_cfg_terminal_reuse_validates - Grammar validation check for reused terminals

All 9 tests pass (4 original + 5 new).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Ki-Seki]

@copilot I have pushed other commits. So you may change the PR desc correspondingly.

[Copilot]

@copilot I have pushed other commits. So you may change the PR desc correspondingly.

Updated the PR description to reflect your changes:

• Documented the docstring update (commit b5d8385) that changed the example from M_i to T_i naming
• Documented the test simplification (commit 4b19946) that consolidated the test cases into a single comprehensive test within test_build_cfg()
• Listed all 6 commits in the PR with proper attribution

[Copilot]

Pull request overview

Improves build_cfg() performance by reusing LLGuidance terminal rules when multiple tags share the same content regex, reducing redundant grammar output (especially for the default /(?s:.*)/ pattern).

Changes:

• Switch terminal naming from per-tag (M_i) to per-unique-pattern (T_i).
• Deduplicate identical regex patterns via a unique_pattern_terminals mapping and reuse terminals across tags.
• Update build_cfg() tests and the build_cfg() docstring example to match the new grammar output.

Reviewed changes

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

File	Description
src/gimkit/dsls.py	Implements terminal reuse/deduplication and updates the docstring example to reflect T_i terminals.
tests/test_dsls.py	Updates expected grammar strings to use T_i terminals and adds a repeated-regex assertion case.