fix(rules): correct mihomo rule name mapping

[cesaryuan]

Summary

• fix mihomo rule-name mapping so proxy groups are not recorded as rules
• preserve existing Surge rule-name behavior when the resolved rule already matches the last chain hop
• centralize collector rule-name generation into a shared helper and add regression coverage

close #67

Testing

• corepack pnpm --filter @neko-master/collector test -- src/database/repositories/traffic-writer.test.ts
• corepack pnpm --filter @neko-master/collector test -- src/modules/stats/stats.service.test.ts
• corepack pnpm --filter @neko-master/collector exec tsc --noEmit

[Copilot]

Pull request overview

Fixes rule-name normalization in the collector so Mihomo/Clash “proxy group” hops are no longer persisted as rule names, while preserving Surge’s existing rule-name semantics when the resolved rule already matches the last chain hop. This also centralizes rule-name generation to a shared helper and adds regression tests.

Changes:

• Introduce a shared buildRuleName helper and reuse it across SQLite writes, ClickHouse writes, and realtime aggregation.
• Correct Mihomo/Clash rule-name derivation to use rule + rulePayload (instead of the last chain hop).
• Add/adjust tests to cover Mihomo mapping + Surge preservation behavior.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
apps/web/components/features/rules/interactive-rule-stats.tsx	Clarifies (in comment) that Gateway-rule proxy-group names match Surge’s stored rule naming behavior.
apps/collector/src/shared/utils/rule-name.ts	Adds centralized helper for consistent rule-name construction across collectors/writers.
apps/collector/src/modules/stats/stats.service.test.ts	Updates chain-flow regression fixture to reflect corrected rule/rulePayload semantics.
apps/collector/src/modules/realtime/realtime.store.ts	Uses shared helper so realtime rule aggregation matches persisted rule naming.
apps/collector/src/modules/collector/batch-buffer.ts	Uses shared helper for rule-count logging (aligning with persisted rule naming).
apps/collector/src/modules/clickhouse/clickhouse.writer.ts	Uses shared helper so ClickHouse writes match SQLite/realtime rule naming.
apps/collector/src/database/repositories/traffic-writer.test.ts	Adds regression coverage for Mihomo mapping + Surge-preservation behavior.
apps/collector/src/database/repositories/traffic-writer.repository.ts	Replaces duplicated rule-name logic with shared helper and fixes rule→proxy mapping to use the resolved rule name.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.