ai-codeindex 0.26.0__tar.gz → 0.26.2__tar.gz

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/.gitignore

@@ -36,3 +36,4 @@ htmlcov/
 
 # Validation reports (generated)
 scripts/validation/reports/*.md
+*.code-workspace

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/CHANGELOG.md

@@ -7,6 +7,34 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.26.2] - 2026-06-22
+
+**Theme**: enrichment-quality patch. `scan-all --ai` was silently writing punt
+text ("I need the file names") as the directory description for leaf dirs and
+caching it as `enrichment: ok`. A/B on a real TS monorepo: 23/50 punts → 0/50.
+No contract change.
+
+> **Upgrade note**: existing `README_AI.md` files enriched by ≤0.26.1 may carry
+> poisoned descriptions stamped `enrichment: ok`, which the cache will not
+> refresh on its own. After upgrading, run **`codeindex scan-all --ai --retry-all`**
+> once to flush and regenerate them.
+
+### Fixed
+
+- **`scan-all --ai` enrichment now feeds each directory its OWN files/symbols, fixing leaf-dir punts** (GH #94): `_enrich_directories_with_ai` built the prompt from `build_safe_subdir_context(child_dirs)` — child *directory* names only — while `build_enrich_prompt` instructed the model "based ONLY on the file names and symbol names above." A leaf dir's content is *files*, not subdirs, so the prompt carried nothing to describe; worse, a single uninformative child like `__tests__` made the subdir context non-empty and short-circuited the README fallback. On a TypeScript monorepo this hit 21/50 enriched dirs (`controllers`, `services`, `routes`, `screens`, `pages` — all the leaf code dirs), and the model correctly punted ("I need the file names and symbol names"). New `cli_scan._build_enrich_summary` re-parses the directory's own files **non-recursively** (`scan_directory(recursive=False)` — no subtree leak) and runs them through the existing `extract_symbol_summary`, then `enricher.merge_enrich_context` combines that (primary) with the subdir names (supplementary). Both sources are AST/tree-derived, never README markdown, so the anti-injection-chain property `build_safe_subdir_context` was built for is preserved. `build_enrich_prompt`'s instruction was de-lied ("based ONLY on the information above (file names, symbol names, and subdirectories)") and a dir with genuinely no indexable content is now marked `failed (reason: no indexable content)` — retryable — instead of being sent a self-contradictory prompt. *(The original issue blamed `parallel_workers=16` concurrency; that was a confounded experiment — enrichment is a serial loop and the punts occurred serially. Diagnosis corrected by source read + clean repro before this fix.)*
+- **Punt detection (`looks_like_refusal`) now covers the leaf-dir phrasings it was missing** (GH #94, extends GH #85): the GH #85 refusal table had `i need more` but not `i need specific`, and **zero Chinese prefixes** — so `I need specific file names…` / `我需要…文件名和符号名` / `请提供…` slipped through and were stamped `<!-- enrichment: ok -->`, poisoning the cache (skipped on re-run without `--retry-all`). Added `i need specific` plus a deliberately specific Chinese set (`我需要`, `请提供`, `需要具体`, `信息不足`, `我看到的信息`) — kept narrow so legit one-liners that merely contain `需要` (e.g. "需要登录的接口") are not misclassified, locked by `test_chinese_descriptions_with_xuyao_are_not_refusals`.
+
+## [0.26.1] - 2026-06-05
+
+**Theme**: GitFlow + non-Python UX patch. Four bug fixes from continued fabricOS dogfood — all paths where `codeindex` silently no-op'd or warned about state that wasn't real. No contract changes.
+
+### Fixed
+
+- **`codeindex hooks install post-commit` now warns when `.codeindex.yaml` disables the hook** (GH #87): `codeindex init` ships the yaml with `hooks.post_commit.enabled: false` by default, so the installed `.git/hooks/post-commit` wrapper would check the flag at runtime and silently no-op. The install command printed ✓, the user committed code, and `README_AI.md` never updated — classic "installed but not working" trap. New `_maybe_warn_post_commit_disabled` helper reads the yaml after install; when `post_commit.enabled` is false (or absent), prints an actionable reminder with the exact yaml snippet to flip. No behavior change to the install itself — patch-safe. (The doubly-opt-in default itself is a separate contract question deliberately deferred from this patch; the reminder is enough to unblock the first-time path.)
+- **`codeindex init` no longer false-flags installed TS/JS parsers as missing + install hint matches the documented pipx path** (GH #86): `init_wizard.PARSER_PACKAGES` enumerated only python/php/java even though scanner gained TS/JS parsers (#73) and the parser modules exist under `src/codeindex/parsers/{typescript,javascript}/`. Result on a TS-only project: `init --yes` printed `Warning: Missing parsers for: javascript, typescript` despite `pipx inject ai-codeindex tree-sitter-{typescript,javascript}` confirming the packages were already installed. Worse, the hint said `Install with: pip install ai-codeindex[...]` — contradicts the entire documented install path (`pipx install ai-codeindex` everywhere else in CLAUDE.md, README, hooks/index SKILL) and doesn't even work cleanly for a pipx-managed env. Fix: extend `PARSER_PACKAGES` to include typescript/javascript; change the install hint to `pipx inject ai-codeindex tree-sitter-<lang> [tree-sitter-<lang2>...]` (matches the rest of the doc surface). New structural test `test_parser_package_map_covers_scanner_supported_set` locks the invariant — adding a language to scanner.py without exposing it via `PARSER_PACKAGES` will fail the test, preventing the GH #86 drift class from recurring (same pattern as the #73 structural guard).
+- **`scan-all --ai` no longer caches AI refusal text as `enrichment: ok`** (GH #85): when the configured AI backend declined to answer for a directory (e.g. haiku returning "I don't see any file names or symbol names provided…"), Phase 2 wrote the refusal verbatim into the directory's `README_AI.md` blockquote AND stamped `<!-- enrichment: ok -->`. Subsequent `scan-all --ai` runs saw the `ok` marker, skipped re-enrichment, and the garbage description stuck in the cache forever. New `enricher.looks_like_refusal(text)` helper detects common refusal prefixes (case-insensitive: `I don't`, `I cannot`, `I'm unable`, `no file`, `insufficient context`, `as an AI`, etc.); when matched, the directory is marked `failed (reason: ai-refused)` so the next run retries automatically (idempotent re-enrichment policy already in place). Refusal-pattern detection happens on the cleaned/truncated AI output (~80 chars) — refusal phrasing always lives in the first ~30 chars so truncation doesn't blind the check.
+- **Post-commit hook now fires on merge commits** (GH #84): the shell wrapper's loop guard used `git diff-tree --no-commit-id --name-only -r HEAD`, which returns **empty on merge commits** by default — `-m` is required to enumerate per-parent changes. Empty → `NON_DOC_FILES` empty → wrapper exits 0 without delegating to `codeindex hooks run post-commit`. Net effect on GitFlow projects: `README_AI.md` files never auto-updated on PR merges (the commits that actually bring new code to `main` / `develop`). Reported live on fabricOS HEAD `171702b` (Merge PR #7). Fix: add `-m` to the diff-tree command. Verified against a real merge commit in this repo (`e240ba0` — Merge develop for 0.26.0 release): old logic saw 0 files, new logic sees the 20 files actually changed. New regression test in `tests/test_cli_hooks.py::TestHookGeneration::test_post_commit_loop_guard_handles_merge_commits` locks both directions (rejects the broken pattern, requires either `-m` or `git show --name-only`).
+
 ## [0.26.0] - 2026-06-01
 
 **Theme**: dogfood-driven CLI bug sweep. Five user-reported issues from fabricOS dogfood — mostly UX cliffs in `init` and `list-dirs` on non-Python projects, plus a navigation-correctness bug in nav-level `README_AI.md`. One contract change in `init --yes` so first `scan-all --ai` works out of the box.

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ai-codeindex
-Version: 0.26.0
+Version: 0.26.2
 Summary: AI-native code indexing tool for large codebases
 Project-URL: Homepage, https://github.com/dreamlx/codeindex
 Project-URL: Documentation, https://github.com/dreamlx/codeindex
@@ -578,7 +578,7 @@ See [Release Automation Guide](docs/development/QUICK_START_RELEASE.md) for deta
 
 ## Roadmap
 
-**Current version**: v0.25.0
+**Current version**: v0.26.2
 
 **Recent milestones**:
 - v0.23.0 — **AI-Enhanced Module Descriptions**: two-phase pipeline, auto-AI enrichment, post-commit thin wrapper

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/README.md

@@ -521,7 +521,7 @@ See [Release Automation Guide](docs/development/QUICK_START_RELEASE.md) for deta
 
 ## Roadmap
 
-**Current version**: v0.25.0
+**Current version**: v0.26.2
 
 **Recent milestones**:
 - v0.23.0 — **AI-Enhanced Module Descriptions**: two-phase pipeline, auto-AI enrichment, post-commit thin wrapper

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/docs/planning/ROADMAP.md

@@ -1,7 +1,7 @@
 # codeindex Strategic Roadmap
 
 **Last Updated**: 2026-03-06
-**Current Version**: v0.25.0
+**Current Version**: v0.26.2
 **Vision**: Universal Code Parser - Best-in-class multi-language AST parser for AI-assisted development
 **Positioning**: Focused on code parsing and structured data extraction, not AI analysis
 
@@ -534,4 +534,4 @@
 **Next Review**: 2026-03-31
 **Maintained By**: @dreamlx
 **Last Updated**: 2026-03-06
-**Current Version**: v0.25.0
+**Current Version**: v0.26.2

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ai-codeindex"
-version = "0.26.0"
+version = "0.26.2"
 description = "AI-native code indexing tool for large codebases"
 readme = "README.md"
 requires-python = ">=3.10"

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/src/codeindex/README_AI.md

@@ -1,12 +1,12 @@
 <!-- codeindex navigation index — agent: drill into source via Read/Grep for precise mechanism; do not treat this as final word. -->
-<!-- Generated by codeindex (detailed) at 2026-06-01T11:56:26.178371 -->
+<!-- Generated by codeindex (detailed) at 2026-06-21T23:56:14.087118 -->
 
 # codeindex
 
 ## Overview
 
 - **Files**: 89
-- **Symbols**: 570
+- **Symbols**: 574
 
 ## Files
 
@@ -162,7 +162,7 @@ This module provides:
 - `def install_hook(hook_name: str, repo_path: Optional[Path] = None) -> bool`
 - `def uninstall_hook(hook_name: str, repo_path: Optional[Path] = None) -> bool`
 
-_... and 6 more symbols_
+_... and 7 more symbols_
 
 ### cli_parse.py
 _CLI parse command - Parse a single source file and output JSON.
@@ -227,6 +227,11 @@ s_
     timeout: int = 120,
     retry_all: bool = False,
 ) -> None`
+- `def _build_enrich_summary(
+    dir_path: Path,
+    child_dirs: list[Path],
+    config: Config,
+) -> str`
 - `def _enrich_directories_with_ai(
     dirs: list[Path],
     tree: DirectoryTree,
@@ -237,14 +242,8 @@ s_
     enrichment_cache: dict[Path, str] | None = None,
 ) -> None`
 - `def _print_enrichment_failure_hint(failed_dirs: list[str]) -> None`
-- `def _process_directory_with_smartwriter(
-    dir_path: Path,
-    tree: DirectoryTree,
-    config: Config,
-    docstring_processor=None,
-) -> tuple[Path, bool, str, int]`
 
-_... and 2 more symbols_
+_... and 3 more symbols_
 
 ### cli_symbols.py
 _CLI commands for symbol indexing and dependency analysis.
@@ -345,7 +344,8 @@ co_
 - `def show_full_config_help() -> None`
 - `def _show_param_section(param_name: str) -> None`
 - `def explain_parameter(
-    param_name: str, curr
+    param_name: str, current_value: Optional[any] = None, cpu_count: Optional[int] = None
+
 
 ---
 _Content truncated due to size limit. See individual module README files for details._

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/src/codeindex/cli_hooks.py

@@ -316,8 +316,11 @@ exit 0
 # Post-commit hook for codeindex
 # Thin wrapper — all logic in Python (auto-updated via pip)
 
-# Avoid infinite loop: skip if last commit only contains README_AI.md
-LAST_COMMIT_FILES=$(git diff-tree --no-commit-id --name-only -r HEAD)
+# Avoid infinite loop: skip if last commit only contains README_AI.md.
+# -m is required so merge commits enumerate per-parent changes — without it
+# `git diff-tree -r HEAD` returns empty on every merge commit and the hook
+# silently skips every PR merge in a GitFlow project (GH #84).
+LAST_COMMIT_FILES=$(git diff-tree --no-commit-id --name-only -r -m HEAD)
 NON_DOC_FILES=$(echo "$LAST_COMMIT_FILES" | \\
     grep -v "README_AI.md" | grep -v "PROJECT_INDEX.md" || true)
 if [ -z "$NON_DOC_FILES" ]; then
@@ -539,6 +542,48 @@ def hooks():
     pass
 
 
+def _maybe_warn_post_commit_disabled(project_dir: Path) -> None:
+    """Print a reminder if ``hooks.post_commit.enabled`` is false in
+    ``.codeindex.yaml``.
+
+    ``codeindex init`` ships the yaml with ``enabled: false`` by default,
+    so the installed ``.git/hooks/post-commit`` wrapper no-ops at runtime
+    even though the install command printed ✓. User commits, READMEs don't
+    update, "it doesn't work" — see GH #87.
+
+    We don't flip the flag automatically (contract change, see #75 for the
+    boundary). The reminder makes the contract visible at install time so
+    the user can decide.
+    """
+    yaml_path = project_dir / ".codeindex.yaml"
+    if not yaml_path.exists():
+        return
+
+    try:
+        import yaml
+
+        with open(yaml_path) as f:
+            data = yaml.safe_load(f) or {}
+    except Exception:
+        # Yaml unreadable or parse error — silent skip; the install itself
+        # succeeded, we don't want this advisory to mask the real result.
+        return
+
+    enabled = data.get("hooks", {}).get("post_commit", {}).get("enabled", False)
+    if enabled:
+        return
+
+    console.print(
+        "[yellow]⚠[/yellow]  [bold]post_commit.enabled is false in "
+        ".codeindex.yaml[/bold] — the installed hook wrapper checks this "
+        "flag at runtime and will no-op until you flip it.\n"
+        "   Edit [cyan].codeindex.yaml[/cyan]:\n"
+        "     hooks:\n"
+        "       post_commit:\n"
+        "         [bold]enabled: true[/bold]\n"
+    )
+
+
 @hooks.command()
 @click.option(
     "--all",
@@ -625,6 +670,13 @@ def install(hook_name: Optional[str], install_all: bool, force: bool):
                 f"[dim]→ Skipped {skipped_count} already installed hook(s)[/dim]\n"
             )
 
+        # GH #87: surface the runtime-disabled trap. If post-commit was just
+        # installed (or was already installed), the wrapper still no-ops when
+        # .codeindex.yaml has post_commit.enabled=false (the init default).
+        # Without this reminder, install prints ✓ but commits trigger nothing.
+        if "post-commit" in hooks_to_install:
+            _maybe_warn_post_commit_disabled(Path.cwd())
+
     except ValueError as e:
         console.print(f"[red]✗[/red] Error: {e}", style="red")
         raise click.Abort()

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/src/codeindex/cli_scan.py

@@ -634,6 +634,47 @@ def _process_directories_parallel(
         )
 
 
+def _build_enrich_summary(
+    dir_path: Path,
+    child_dirs: list[Path],
+    config: Config,
+) -> str:
+    """Build the enrichment prompt context for one directory (GH #94).
+
+    Combines the directory's OWN file/symbol summary — re-parsed from source
+    (AST), non-recursively, so subtree content does not leak in — with its
+    subdirectory names. The dir's own content is the primary signal; subdir
+    names are supplementary. This fixes leaf dirs (e.g. ``controllers`` whose
+    only child is ``__tests__``) that previously got only a useless
+    ``Subdirectories: __tests__`` and made the model punt.
+
+    Args:
+        dir_path: Directory being enriched.
+        child_dirs: Indexed child directories (from ``tree.get_children``).
+        config: Active configuration (for language/exclude filtering).
+
+    Returns:
+        Combined context string, or empty if the dir has no indexable content.
+    """
+    from .enricher import (
+        build_safe_subdir_context,
+        extract_symbol_summary,
+        merge_enrich_context,
+    )
+    from .parallel import parse_files_parallel
+    from .scanner import scan_directory
+
+    subdir_context = build_safe_subdir_context(child_dirs)
+
+    own_summary = ""
+    scan_result = scan_directory(dir_path, config, recursive=False)
+    if scan_result.files:
+        parse_results = parse_files_parallel(scan_result.files, config, quiet=True)
+        own_summary = extract_symbol_summary(parse_results)
+
+    return merge_enrich_context(own_summary, subdir_context)
+
+
 def _enrich_directories_with_ai(
     dirs: list[Path],
     tree: DirectoryTree,
@@ -665,9 +706,8 @@ def _enrich_directories_with_ai(
     """
     from .enricher import (
         build_enrich_prompt,
-        build_safe_subdir_context,
-        extract_summary_from_readme,
         inject_blockquote,
+        looks_like_refusal,
         mark_enrichment_status,
         should_enrich,
     )
@@ -714,17 +754,21 @@ def _enrich_directories_with_ai(
     for dir_path in enrich_dirs:
         readme_path = dir_path / config.output_file
 
-        # Prefer tree-derived context (subdir names only) to avoid the
-        # injection chain through markdown-embedded AI descriptions.
-        # Fall back to README parsing only for dirs without indexed children
-        # (rare for enrichable navigation/overview dirs).
+        # Build context from the dir's OWN files/symbols (primary) plus its
+        # subdirectory names (supplementary). Both come from AST/tree sources,
+        # never README markdown, so the anti-injection-chain property holds
+        # (GH #94).
         child_dirs = tree.get_children(dir_path)
-        summary = build_safe_subdir_context(child_dirs)
-
-        if not summary:
-            summary = extract_summary_from_readme(readme_path)
+        summary = _build_enrich_summary(dir_path, child_dirs, config)
 
         if not summary:
+            # Nothing indexable to describe — mark failed (retryable) rather
+            # than send a prompt that asks the model to describe nothing.
+            if readme_path.exists():
+                mark_enrichment_status(
+                    readme_path, "failed", reason="no indexable content"
+                )
+            failed_dirs.append(dir_path.name)
             continue
 
         # Include parent directory name for context
@@ -759,6 +803,20 @@ def _enrich_directories_with_ai(
             failed_dirs.append(dir_path.name)
             continue
 
+        # AI returned text, but is it an actual description or a refusal?
+        # Stamping refusal text with `enrichment: ok` poisons the cache forever
+        # because subsequent runs see the marker and skip re-enrichment (GH #85).
+        if looks_like_refusal(description):
+            if readme_path.exists():
+                mark_enrichment_status(readme_path, "failed", reason="ai-refused")
+            failed_dirs.append(dir_path.name)
+            if not quiet:
+                console.print(
+                    f"[yellow]⚠[/yellow] {dir_path.name}: AI declined "
+                    f"(refusal pattern) — will retry next `scan-all --ai`"
+                )
+            continue
+
         # Inject into README_AI.md
         if readme_path.exists():
             inject_blockquote(readme_path, description)

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/src/codeindex/enricher.py

@@ -18,6 +18,54 @@ _MAX_SYMBOLS_PER_FILE = 5
 # Maximum total files to include in the prompt
 _MAX_FILES = 15
 
+# Refusal prefixes used to detect AI responses that decline to summarise the
+# directory (typically because the prompt context was thin enough that the
+# model can't say anything specific). When the model returns "I don't see any
+# file names..." we must NOT stamp `enrichment: ok` — that poisons the cache
+# permanently (GH #85). Instead the response is treated as a failure with
+# reason "ai-refused" so the next `scan-all --ai` retries automatically.
+#
+# Matched against the lowercased, stripped response. Keep prefixes short so
+# the check is robust to upstream truncation (the cli_scan path truncates
+# enrichment output to ~80 chars; refusal phrasing is always in the first 30).
+_REFUSAL_PREFIXES = (
+    "i don't",
+    "i do not",
+    "i cannot",
+    "i can't",
+    "i'm unable",
+    "i am unable",
+    "i need more",
+    "i need specific",  # GH #94: leaf-dir punt asking for omitted file/symbol names
+    "no file",
+    "insufficient context",
+    "without more",
+    "sorry, i",
+    "as an ai",
+    # Chinese punts (GH #94). Kept specific so legit one-liners that merely
+    # contain 需要 (e.g. "需要登录的接口") are not misclassified.
+    "我需要",
+    "请提供",
+    "需要具体",
+    "信息不足",
+    "我看到的信息",
+)
+
+
+def looks_like_refusal(text: str) -> bool:
+    """Return True if the cleaned AI response looks like a refusal / inability.
+
+    Used by ``scan-all --ai`` Phase 2 to distinguish a usable one-line
+    description from boilerplate refusal text. The refusal text is never
+    project-information and stamping it with ``<!-- enrichment: ok -->``
+    pins it in the cache; the next run sees the marker and skips
+    re-enrichment, so the garbage description sticks indefinitely (GH #85).
+    """
+    t = (text or "").strip().lower()
+    if not t:
+        return False
+    return any(t.startswith(p) for p in _REFUSAL_PREFIXES)
+
 
 def extract_symbol_summary(parse_results: list[ParseResult]) -> str:
     """Extract a compact summary of file names + symbol names for AI prompt.
@@ -125,6 +173,35 @@ def build_safe_subdir_context(child_dirs: list[Path]) -> str:
     return "Subdirectories: " + ", ".join(names) + suffix
 
 
+def merge_enrich_context(own_summary: str, subdir_context: str) -> str:
+    """Combine a directory's own file/symbol summary with its subdir names.
+
+    GH #94: a directory's OWN files and symbols are the primary signal for a
+    one-line description; subdirectory names are only supplementary. The
+    previous enrichment path used subdir context with a README fallback and a
+    single uninformative child (e.g. ``__tests__``) made the subdir context
+    non-empty, short-circuiting the dir's own content and leaving leaf dirs
+    with an empty prompt — so the model punted ("I need the file names").
+
+    Both inputs come from AST/tree sources (``extract_symbol_summary`` and
+    ``build_safe_subdir_context``), never from README markdown, preserving the
+    anti-injection-chain property ``build_safe_subdir_context`` was built for.
+
+    Args:
+        own_summary: ``extract_symbol_summary`` output (file: symbols; ...).
+        subdir_context: ``build_safe_subdir_context`` output (Subdirectories: ...).
+
+    Returns:
+        Combined context string, or empty string if both inputs are empty.
+    """
+    parts = []
+    if own_summary and own_summary.strip():
+        parts.append(f"Files: {own_summary.strip()}")
+    if subdir_context and subdir_context.strip():
+        parts.append(subdir_context.strip())
+    return "\n".join(parts)
+
+
 def build_enrich_prompt(
     dir_name: str,
     symbol_summary: str,
@@ -148,13 +225,14 @@ def build_enrich_prompt(
     return (
         context
         + "\n"
-        "Based ONLY on the file names and symbol names above, write a concise "
-        "functional description of this module (30 chars or less). "
+        "Based ONLY on the information above (file names, symbol names, and "
+        "subdirectories), write a concise functional description of this "
+        "module (30 chars or less). "
         "Describe WHAT it does, not HOW. "
         "Examples: '会员等级、积分、权益卡管理', 'Payment gateway (Alipay/WeChat)', "
         "'物流配送与运费计算'. "
         "Output ONLY the description text. No quotes, no markdown, no explanation. "
-        "Do NOT invent features not evidenced by the symbol names."
+        "Do NOT invent features not evidenced by the names above."
     )
 
 

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/src/codeindex/init_wizard.py

@@ -94,11 +94,19 @@ FRAMEWORK_PATTERNS: Dict[str, Dict[str, str]] = {
 
 
 
-# Parser package mapping: language -> tree-sitter package name
+# Parser package mapping: language -> tree-sitter Python import name.
+#
+# Must stay in sync with ``scanner.LANGUAGE_EXTENSIONS``. The structural
+# test ``test_parser_package_map_covers_scanner_supported_set`` locks the
+# invariant — adding a language to scanner.py without exposing it here
+# silently re-introduces the GH #86 false-positive ("missing parsers" for
+# parsers that are actually installed).
 PARSER_PACKAGES = {
     "python": "tree_sitter_python",
     "php": "tree_sitter_php",
     "java": "tree_sitter_java",
+    "typescript": "tree_sitter_typescript",
+    "javascript": "tree_sitter_javascript",
 }
 
 
@@ -146,8 +154,13 @@ def get_parser_install_guidance(languages: list[str]) -> dict:
     }
 
     if missing:
-        extras = ",".join(missing)
-        result["install_command"] = f"pip install ai-codeindex[{extras}]"
+        # Use `pipx inject ai-codeindex tree-sitter-<lang>` to match the
+        # documented install path (CLAUDE.md, README, hooks/index SKILL all
+        # use pipx). The previous `pip install ai-codeindex[<lang>]` hint
+        # contradicted the recommended path AND broke for pipx-managed
+        # envs — see GH #86 (4b).
+        pkgs = " ".join(f"tree-sitter-{lang}" for lang in missing)
+        result["install_command"] = f"pipx inject ai-codeindex {pkgs}"
 
     return result
 

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/tests/README_AI.md

@@ -1,12 +1,12 @@
 <!-- codeindex navigation index — agent: drill into source via Read/Grep for precise mechanism; do not treat this as final word. -->
-<!-- Generated by codeindex (detailed) at 2026-06-01T11:59:55.553144 -->
+<!-- Generated by codeindex (detailed) at 2026-06-21T23:56:14.360118 -->
 
 # tests
 
 ## Overview
 
 - **Files**: 125
-- **Symbols**: 2148
+- **Symbols**: 2172
 
 ## Files
 

{ai_codeindex-0.26.0 → ai_codeindex-0.26.2}/tests/test_cli_hooks.py

@@ -223,6 +223,40 @@ class TestHookGeneration:
 
         assert "codeindex-managed hook" in script
 
+    def test_post_commit_loop_guard_handles_merge_commits(self):
+        """Regression for GH #84.
+
+        ``git diff-tree --no-commit-id --name-only -r HEAD`` returns empty
+        on merge commits by default — it needs ``-m`` to enumerate the
+        per-parent changes. Without the flag, the loop guard sees zero
+        files on every merge commit, hits the "all docs" early-exit, and
+        the wrapper never delegates to ``codeindex hooks run post-commit``.
+
+        Net effect on GitFlow projects: ``README_AI.md`` files never
+        auto-update on PR merges (the commits that bring new code to
+        ``main`` / ``develop``). Reported live on fabricOS HEAD ``171702b``.
+        """
+        script = generate_hook_script("post-commit")
+
+        # The broken pattern: -r HEAD without -m. Asserted as a literal
+        # substring so future whitespace/format tweaks don't accidentally
+        # let it back in.
+        assert "diff-tree --no-commit-id --name-only -r HEAD" not in script, (
+            "Post-commit loop guard uses `git diff-tree -r HEAD` without -m. "
+            "On merge commits this returns empty and the hook silently skips "
+            "(GH #84). Use `-m` or `git show --name-only`."
+        )
+
+        # The fix: either -m on diff-tree OR git show --name-only. Both
+        # produce non-empty output on merge commits.
+        has_dash_m = "diff-tree" in script and " -m " in script
+        has_git_show = "git show --name-only" in script
+        assert has_dash_m or has_git_show, (
+            "Post-commit hook must enumerate files on merge commits. "
+            "Expected either `git diff-tree ... -m HEAD` or "
+            "`git show --name-only ... HEAD` in the loop guard (GH #84)."
+        )
+
 
 class TestBackupAndRestore:
     """Test backup and restore functionality."""
@@ -297,3 +331,85 @@ class TestCLIIntegration:
         """Should provide hooks status CLI command."""
         # This will be implemented with Click
         pass
+
+
+class TestPostCommitEnabledWarning:
+    """GH #87 — ``codeindex hooks install post-commit`` must surface the
+    runtime-disabled trap.
+
+    ``codeindex init`` writes ``.codeindex.yaml`` with
+    ``hooks.post_commit.enabled: false`` by default. Without a reminder at
+    install time, users see the install ✓, make commits, and nothing
+    happens because the wrapper checks the flag at runtime. These tests
+    cover the three meaningful states (disabled / enabled / no yaml)."""
+
+    def _yaml(self, enabled: bool) -> str:
+        return (
+            "version: 1\n"
+            "hooks:\n"
+            "  post_commit:\n"
+            f"    enabled: {str(enabled).lower()}\n"
+        )
+
+    def test_install_warns_when_yaml_disables_post_commit(self, tmp_path):
+        from click.testing import CliRunner
+
+        from codeindex.cli import main
+
+        (tmp_path / ".git" / "hooks").mkdir(parents=True)
+        (tmp_path / ".codeindex.yaml").write_text(self._yaml(enabled=False))
+
+        runner = CliRunner()
+        original_cwd = os.getcwd()
+        try:
+            os.chdir(tmp_path)
+            result = runner.invoke(main, ["hooks", "install", "post-commit"])
+        finally:
+            os.chdir(original_cwd)
+
+        assert result.exit_code == 0, result.output
+        assert "post_commit.enabled is false" in result.output, result.output
+        # The user-visible fix (the yaml snippet) must be present.
+        assert "enabled: true" in result.output
+
+    def test_install_silent_when_yaml_enables_post_commit(self, tmp_path):
+        from click.testing import CliRunner
+
+        from codeindex.cli import main
+
+        (tmp_path / ".git" / "hooks").mkdir(parents=True)
+        (tmp_path / ".codeindex.yaml").write_text(self._yaml(enabled=True))
+
+        runner = CliRunner()
+        original_cwd = os.getcwd()
+        try:
+            os.chdir(tmp_path)
+            result = runner.invoke(main, ["hooks", "install", "post-commit"])
+        finally:
+            os.chdir(original_cwd)
+
+        assert result.exit_code == 0, result.output
+        # No reminder when the user has explicitly enabled it.
+        assert "post_commit.enabled is false" not in result.output
+
+    def test_install_silent_when_no_yaml(self, tmp_path):
+        """Without ``.codeindex.yaml`` we don't know the project's intent;
+        the install itself succeeds, advisory stays silent — matches the
+        rule "don't mask the real result of a non-yaml-driven command"."""
+        from click.testing import CliRunner
+
+        from codeindex.cli import main
+
+        (tmp_path / ".git" / "hooks").mkdir(parents=True)
+        # No .codeindex.yaml.
+
+        runner = CliRunner()
+        original_cwd = os.getcwd()
+        try:
+            os.chdir(tmp_path)
+            result = runner.invoke(main, ["hooks", "install", "post-commit"])
+        finally:
+            os.chdir(original_cwd)
+
+        assert result.exit_code == 0, result.output
+        assert "post_commit.enabled is false" not in result.output