@shuji-bonji/rxjs-mcp 0.3.0 → 0.4.1

package/CHANGELOG.md

@@ -5,6 +5,36 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.1] - 2026-05-17
+
+### Changed
+- **`detect_memory_leak`**: 単純な subscribe/unsubscribe カウント比較を廃止し、`takeUntilDestroyed()`（Angular 16+）/ `take(N)` / `first()` / `firstValueFrom` / `useEffect` cleanup / `onUnmounted` 等の **自動クリーンアップパターンを認識** するように改修。これにより、Angular 16+ の `takeUntilDestroyed()` 等を使った現代的なコードに対する false positive を解消。
+- **`detect_memory_leak`** の description に "Recognizes modern auto-cleanup patterns" を明記。
+- **memory-leak と lint_rxjs の検出ロジック共通化**: `src/shared/subscription-analysis.ts` を新設し、サブスクリプション解析のヒューリスティクスを一元化。
+
+### Fixed
+- **`getWorkerPath()` の Windows 非互換**: `__dirname.includes('/src/')` というハードコードを `path.sep` 経由のセグメント走査に変更。Windows (`\src\`) でも src/dist 判定が正しく動作するように。
+- **`marble-diagram.ts`**: 未使用変数 `index` 削除、case block の `const value` をブロック化、内部ヘルパー `parseMarbleSyntax` の名前を `_parseMarbleSyntax`（未使用許可・将来の marble syntax 入力対応用）に整理。
+
+### Infrastructure
+- **ESLint 自己適用**: `eslint.config.js`（flat config）を追加し、`@eslint/js` + `typescript-eslint` を全 `src/` に、`eslint-plugin-rxjs-x` を `execute-stream*.ts` のみに（type-aware）適用。`npm run lint` / `npm run lint:fix` を追加し、CI workflow にも `Lint` ステップを追加。
+- **Dependabot**: `.github/dependabot.yml` を追加。npm 依存（dev/prod グループ分け、週次）と GitHub Actions（月次）を自動更新。
+- **MCP integration test の vitest 化**: `src/tools/mcp-integration.test.ts` を新規追加。子プロセスとして `dist/index.js` を spawn し、JSON-RPC で全 6 ツールを実機テスト。`npm test` で一括実行可能（dist 未ビルド時は自動 skip）。既存の `test-mcp-server.mjs` は legacy 用に残存（`npm run test:mcp`）。
+
+## [0.4.0] - 2026-05-17
+
+### Added
+- **`lint_rxjs` ツール**: eslint-plugin-rxjs-x の recommended/strict ルールを正規表現ベースで再実装。コードスニペットを渡すだけで即座にフィードバック（ESLint 不要）
+  - **20 recommended ルール**: no-async-subscribe, no-create, no-nested-subscribe, no-sharereplay, prefer-root-operators, no-topromise, no-unsafe-takeuntil, throw-error 等
+  - **8 strict ルール**: no-exposed-subjects, no-floating-observables, no-misused-observables, no-subclass, finnish 等
+  - **フレームワーク固有ルール**: Angular (takeUntilDestroyed 推奨), React (useEffect 内 subscribe), Vue (onUnmounted cleanup)
+  - `config` パラメータ: `recommended`（デフォルト）/ `strict`
+  - `framework` パラメータ: `angular` / `react` / `vue` / `none`
+  - `rules` パラメータ: 個別ルール指定可能
+
+### Changed
+- `DOC_BASE_URL` エイリアスを削除 (v0.3.0 で deprecated 告知済み)
+
 ## [0.3.0] - 2026-05-16
 
 ### BREAKING

package/README.ja.md

@@ -62,6 +62,12 @@ ClaudeなどのAIアシスタントから直接RxJSストリームを実行、
   - 状態管理
   - その他多数...
 
+### 🔍 RxJS Lint（v0.4.0〜）
+- eslint-plugin-rxjs-x ルールの正規表現ベース再実装
+- ESLint / TypeScript パーサー不要で即チェック
+- recommended（20ルール）/ strict（28ルール）の2段階
+- Angular / React / Vue 固有のフレームワークルール対応
+
 ## インストール
 
 ```bash
@@ -197,6 +203,31 @@ source$.pipe(
 - `cache-refresh` - リフレッシュ戦略付きキャッシュ
 - その他多数...
 
+### lint_rxjs
+
+RxJSコードスニペットを静的解析し、一般的な問題とベストプラクティスを検出します。[eslint-plugin-rxjs-x](https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x) のルールを正規表現ベースで再実装しています（ESLint不要）。
+
+```typescript
+// パラメータ:
+{
+  code: string;           // 解析対象の RxJS コード
+  config?: 'recommended' | 'strict';  // ルールセット（デフォルト: recommended）
+  framework?: 'angular' | 'react' | 'vue' | 'none'; // フレームワークコンテキスト
+  rules?: string[];       // 個別ルール指定（config を上書き）
+}
+```
+
+**Config レベル:**
+
+- `recommended` — 最も一般的な問題をカバーする20ルール
+- `strict` — スタイルチェック含む全ルール（finnish, no-exposed-subjects 等）
+
+**フレームワーク固有チェック:**
+
+- **Angular** — コンポーネント内の `takeUntilDestroyed()` / `takeUntil(destroy$)` 欠落を検出
+- **React** — `useEffect` なしの `subscribe()` を検出
+- **Vue** — `onUnmounted` なしの `subscribe()` を検出
+
 ## 使用例
 
 ### Claudeでの使用
@@ -300,6 +331,7 @@ RxJS MCP Serverは以下と組み合わせて使用できます：
 │ • analyze_operators│
 │ • detect_memory_leak│
 │ • suggest_pattern│
+│ • lint_rxjs      │
 └─────────────────┘
 ```
 

package/README.md

@@ -212,6 +212,31 @@ Available patterns:
 - `cache-refresh` - Cache with refresh strategy
 - And more...
 
+### lint_rxjs
+
+Lint RxJS code snippets for common issues and best practices. Based on [eslint-plugin-rxjs-x](https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x) rules.
+
+```typescript
+// Parameters:
+{
+  code: string;           // RxJS code to lint
+  config?: 'recommended' | 'strict';  // Rule set (default: recommended)
+  framework?: 'angular' | 'react' | 'vue' | 'none'; // Framework context
+  rules?: string[];       // Specific rules to check (overrides config)
+}
+```
+
+**Config levels:**
+
+- `recommended` — 20 rules covering the most common issues
+- `strict` — All rules including style checks (finnish, no-exposed-subjects, etc.)
+
+**Framework-specific checks:**
+
+- **Angular** — Detects missing `takeUntilDestroyed()` or `takeUntil(destroy$)` in components
+- **React** — Detects `subscribe()` without `useEffect` cleanup
+- **Vue** — Detects `subscribe()` without `onUnmounted` cleanup
+
 ## Usage Examples
 
 ### With Claude
@@ -329,9 +354,42 @@ Future Meta-MCP integration will allow seamless coordination between these tools
 │ • analyze_operators│
 │ • detect_memory_leak│
 │ • suggest_pattern│
+│ • lint_rxjs      │
 └─────────────────┘
 ```
 
+## Documentation Reference System
+
+Since v0.3.0, `analyze_operators` outputs three-tier documentation links for each operator and creation function:
+
+| Tier | Source | Purpose | AI-readable? |
+|------|--------|---------|:---:|
+| **Official** | [rxjs.dev](https://rxjs.dev) | Authoritative API reference for humans | ❌ (SPA) |
+| **Source** | [GitHub (tag 7.8.2)](https://github.com/ReactiveX/rxjs/tree/7.8.2/src/internal) | JSDoc + implementation — the richest context for AI | ✅ |
+| **Guide** | [RxJS-with-TypeScript](https://github.com/shuji-bonji/RxJS-with-TypeScript) | Bilingual JP/EN explanations with practical examples | ✅ |
+
+### Why include the community guide alongside official docs?
+
+1. **rxjs.dev is a client-rendered SPA.** AI assistants cannot fetch its content — HTTP requests return an empty shell with JavaScript loaders. The official site is therefore a "link to hand to humans," not a source AI can read.
+
+2. **GitHub source provides raw truth.** The RxJS source code (pinned at tag `7.8.2`) contains JSDoc, type signatures, and implementation details. This is the primary reference for AI assistants.
+
+3. **The bilingual guide adds learning context.** It organizes operators by use-case (not just alphabetically), provides runnable examples, and offers Japanese translations. For Japanese-speaking users or learners, this fills a gap that neither rxjs.dev nor raw source addresses.
+
+### Priority order
+
+When the MCP server outputs references, it follows this priority:
+
+1. `officialUrl` — always shown (authority, human-readable)
+2. `sourceUrl` — shown when available (AI should read this)
+3. `guideUrl` — shown when the page exists (supplementary)
+
+If a guide page does not yet exist for an operator, the field is simply omitted (no broken link). Coverage is tracked by the [URL validation CI](.github/workflows/url-validation.yml).
+
+### Can I disable the guide references?
+
+Currently there is no runtime option to exclude `guideUrl` from output. If you prefer official-only references, you can fork this server or open a feature request. A future version may support a `--references=official,source` flag.
+
 ## Contributing
 
 Contributions are welcome! Please feel free to submit a PR.

package/dist/data/lint-rules.d.ts

@@ -0,0 +1,33 @@
+/**
+ * RxJS Lint Rules Database
+ *
+ * Regex-based reimplementation of eslint-plugin-rxjs-x rules.
+ * These rules work on code snippets without requiring ESLint runtime or TypeScript parser.
+ *
+ * Reference: https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x
+ */
+export type LintSeverity = 'error' | 'warning' | 'info';
+export type LintConfig = 'recommended' | 'strict';
+export type FrameworkContext = 'angular' | 'react' | 'vue' | 'none';
+export interface LintDiagnostic {
+    rule: string;
+    severity: LintSeverity;
+    message: string;
+    line?: number;
+    suggestion?: string;
+    docUrl: string;
+}
+export interface LintRule {
+    name: string;
+    description: string;
+    severity: LintSeverity;
+    config: LintConfig;
+    /** Whether this rule requires type information (cannot be fully checked with regex) */
+    requiresTypeInfo: boolean;
+    docUrl: string;
+    check: (code: string, framework: FrameworkContext) => LintDiagnostic[];
+}
+export declare const allLintRules: LintRule[];
+/** Get rules for a given config level */
+export declare function getRulesForConfig(config: LintConfig): LintRule[];
+//# sourceMappingURL=lint-rules.d.ts.map

package/dist/data/lint-rules.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"lint-rules.d.ts","sourceRoot":"","sources":["../../src/data/lint-rules.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,MAAM,MAAM,YAAY,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AACxD,MAAM,MAAM,UAAU,GAAG,aAAa,GAAG,QAAQ,CAAC;AAClD,MAAM,MAAM,gBAAgB,GAAG,SAAS,GAAG,OAAO,GAAG,KAAK,GAAG,MAAM,CAAC;AAEpE,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,YAAY,CAAC;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,QAAQ;IACvB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,YAAY,CAAC;IACvB,MAAM,EAAE,UAAU,CAAC;IACnB,uFAAuF;IACvF,gBAAgB,EAAE,OAAO,CAAC;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,gBAAgB,KAAK,cAAc,EAAE,CAAC;CACxE;AA+/BD,eAAO,MAAM,YAAY,EAAE,QAAQ,EAiClC,CAAC;AAEF,yCAAyC;AACzC,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,UAAU,GAAG,QAAQ,EAAE,CAMhE"}