docsif 1.0.0__tar.gz → 1.0.1__tar.gz

{docsif-1.0.0 → docsif-1.0.1}/.gsd/dispatch-isolation-sentinel.json

@@ -1 +1 @@
-{"isolation":"none","harness_flag":null,"phase":null,"plan":null,"written_at":1793973385395}
+{"isolation":"none","harness_flag":null,"phase":null,"plan":null,"written_at":1794012884815}

docsif-1.0.1/.planning/.continue-here.md

@@ -0,0 +1,80 @@
+---
+context: default
+phase: none
+task: 0
+total_tasks: 0
+status: milestone_shipped
+last_updated: 2026-11-06T23:17:56.807Z
+---
+
+# BLOCKING CONSTRAINTS — Read Before Anything Else
+
+> These are not suggestions. Each constraint below was discovered through failure.
+> Acknowledge each one explicitly before proceeding.
+
+- [ ] CONSTRAINT: generator-owned docs are version-sensitive by construction — any change to generator behavior or the libraries they introspect (Click majors, Python minors, pydantic) re-drifts `docs/cli-reference.md`/`configuration.md`; after touching a generator or bumping an introspected dep, ALWAYS run `make docs-generate` and commit the regenerated output before push, or CI's `git diff --exit-code` gate fails.
+
+**Do not proceed until the box is checked.**
+
+## Critical Anti-Patterns
+
+| Pattern | Description | Severity | Prevention Mechanism |
+|---------|-------------|----------|---------------------|
+| Floors-only unpinned deps | CI installs latest deps (Click 8.5 vs local 8.3) and behavior diverged, failing the drift gate; a fresh-install test env also surfaced undeclared pytest-asyncio | advisory | Generator output is now version-normalized; before releases, smoke-test a FRESH venv install (`uv pip install docsif[dev]`) — it catches undeclared/exploded deps the canonical env masks |
+| Post-verification edits re-stale fingerprints | Fixing files covered by an already-verified phase (shared surfaces: get.py, Makefile, docs) invalidates that phase's digest, cascading | advisory | Sequence wide fixes BEFORE verifiers run, or expect one extra regeneration round; check `verification.status` per phase after multi-phase fix sessions |
+
+<current_state>
+Milestone v1.0 is SHIPPED and RELEASED. All planning artifacts archived under `.planning/milestones/v1.0-*`; `.planning/phases/` and `.planning/quick/` are empty; ROADMAP collapsed to a shipped-milestone summary; tag `v1.0` pushed; PR #1 (dev→main) open with green CI; **docsif 1.0.0 live on PyPI** (verified installable). No active phase work.
+</current_state>
+
+<completed_work>
+
+- Milestone v1.0 close: verified_closeout, zero overrides (audit 3/3 resolved, not acknowledged)
+- 4 stale phase verifications regenerated (05/06/07/10.1) — 5 review criticals + 4 verification gaps fixed en route (get.py markup display ×2, generator pipes/version-drift, README command, docs CI `--no-cov`, Mermaid sync, phantom fusion, k1)
+- Version 1.0.0 (6 sites), tag v1.0, archives, MILESTONES.md, RETROSPECTIVE.md
+- Pushed dev (207 commits) + tag; PR #1 created; CI Docs Validation failure diagnosed (Click 8.5 drift) and fixed — green
+- PyPI: published as **docsif 1.0.0** (`sif` taken; user chose their own registered name)
+- numpy≥2: canonical suite (numpy 2.3.5) 1015 green; published wheel on numpy 2.5.4 → CLI smoke + 974/974
+- pytest-asyncio added to dev extras (fresh-env 56-failure root cause)
+</completed_work>
+
+<remaining_work>
+
+No open phase work. Backlog (all ledgered in archived REVIEW-DISPOSITION files):
+- 10-REVIEW WR-01..10, 10.1 IN-10..13, 07 WR-02/IN-02/03, delta WR-02 (pip constraints)/IN-02
+- Version quadruplication sync mechanism; arch-generator CI wiring; Makefile test-fast echo wording; mcp/cli.py 0% shim; `context remove/prune` confirmation guardrails
+- Human: merge PR #1; optionally `rm -rf ~/.claude/skills/docsift-{get,search}` (stale pre-rename dirs)
+</remaining_work>
+
+<decisions_made>
+
+- Distribution name `docsif` on PyPI (import/console name stays `sif`)
+- verified_closeout after resolving everything (user principle: resolve, don't acknowledge)
+- Generator outputs normalized version-insensitively rather than pinning CI deps
+- pytest-asyncio fix rides the next release (dev extras don't affect runtime users)
+</decisions_made>
+
+<blockers>
+- None
+</blockers>
+
+## Required Reading (in order)
+
+1. `.planning/MILESTONES.md` — what v1.0 shipped
+2. `.planning/RETROSPECTIVE.md` — patterns/lessons (display-integrity triple precedent, gate discipline)
+3. `.planning/STATE.md` — current position (Awaiting next milestone)
+
+## Infrastructure State
+
+- Branch `dev` @ d0682ba pushed; PR #1 open (CI green); tag `v1.0` pushed
+- PyPI: docsif 1.0.0 published (pypi.org/project/docsif/1.0.0/) — twine creds in `~/.pypirc` ([pypi] + [testpypi])
+- CI: Docs Validation green (run 42310044731) — triggers on docs/README/cli/config/scripts/test_docs paths
+- Untracked by intent: `.planning/state.json` (runtime contract state), `.planning/tmp/` (transient)
+
+<context>
+Session arc: resume → quality gates → ty debt cleared → docs verified → user directed milestone close with "resolve everything" → 4-phase verification regeneration (with real bug fixes surfaced by each round) → clean closeout → push/PR → CI fix → PyPI release (name collision resolved to docsif) → numpy≥2 verification → dev-extras fix. The release is complete; next session starts fresh at milestone planning.
+</context>
+
+<next_action>
+Start with: `/gsd-new-milestone` (or merge PR #1 first if handling the release flow manually).
+</next_action>

docsif-1.0.1/.planning/HANDOFF.json

@@ -0,0 +1,37 @@
+{
+  "version": "1.0",
+  "timestamp": "2026-11-06T23:17:56.807Z",
+  "phase": null,
+  "phase_name": null,
+  "phase_dir": null,
+  "plan": null,
+  "task": null,
+  "total_tasks": 0,
+  "status": "paused",
+  "completed_tasks": [
+    {"id": 1, "name": "Milestone v1.0 close — verified_closeout (11/11 phases fresh-verified, 31/31 requirements, zero overrides)", "status": "done", "commit": "b63eca1"},
+    {"id": 2, "name": "Four stale-verification regenerations (05/06/07/10.1) incl. 5 review criticals fixed along the way", "status": "done", "commit": "92e0150"},
+    {"id": 3, "name": "Version 1.0.0 bump + tag v1.0 + archives (milestones/v1.0-*) + RETROSPECTIVE", "status": "done", "commit": "11b8640"},
+    {"id": 4, "name": "Push dev + tag; PR #1 dev→main", "status": "done", "commit": "1115409"},
+    {"id": 5, "name": "CI Docs Validation fixed (Click 8.5 vs 8.3 generator divergence normalized) — run 42310044731 green", "status": "done", "commit": "1115409"},
+    {"id": 6, "name": "PyPI release: docsif 1.0.0 published (sif name taken; docsif is the project's own registered name)", "status": "done"},
+    {"id": 7, "name": "numpy>=2 verification: canonical suite 1015 green on numpy 2.3.5; fresh install of PUBLISHED wheel on numpy 2.5.4 → CLI smoke + 974/974 (torch-only module excluded)", "status": "done", "commit": "d0682ba"},
+    {"id": 8, "name": "pytest-asyncio declared in dev extras (fresh-install 56-failure root cause; ships next release)", "status": "done", "commit": "d0682ba"}
+  ],
+  "remaining_tasks": [],
+  "blockers": [],
+  "async_jobs": [],
+  "human_actions_pending": [
+    {"action": "Merge PR #1 (dev → main)", "context": "CI green, v1.0 tagged and published; merge completes the release flow", "blocking": false},
+    {"action": "Optionally remove stale pre-rename skill dirs: rm -rf ~/.claude/skills/docsift-get ~/.claude/skills/docsift-search", "context": "P07-T05: they teach the nonexistent docsift CLI; repo-side sif-* symlinks are correct. Agent home-dir deletion was permission-blocked by design", "blocking": false}
+  ],
+  "decisions": [
+    {"decision": "Distribution name = docsif on PyPI", "rationale": "'sif' taken by an unrelated project; docsif is this account's own registered name (0.1.x-0.2.0). Import name and sif console script unchanged", "phase": "release"},
+    {"decision": "verified_closeout with zero overrides", "rationale": "User chose 'resolve everything': UAT gap live-verified, ResourceWarnings fixed (17→0 warnings), mypy ledger resolved, then 4 stale phases regenerated to fresh fingerprints", "phase": "close"},
+    {"decision": "CLI-reference generator normalized against Click version drift", "rationale": "CI installs unpinned Click (8.5) vs local 8.3: flag defaults (None vs False) and help indentation diverged, failing the drift gate. cleandoc + flag-default canonicalization make output version-insensitive", "phase": "release"},
+    {"decision": "pytest-asyncio added to dev extras rather than republishing 1.0.0", "rationale": "dev extras do not affect runtime installs; the gap only breaks fresh contributor envs; ships with the next version", "phase": "release"}
+  ],
+  "uncommitted_files": ["?? .planning/state.json", "?? .planning/tmp/"],
+  "next_action": "/gsd-new-milestone to plan the next version (v1.1 candidates: LLM query expansion backlog item, open review ledgers WR-01..10/IN-10..13, version-sync mechanism, arch-generator CI wiring). Or merge PR #1 first.",
+  "context_notes": "Milestone v1.0 fully shipped: 11/11 phases verified-closeout, archived under .planning/milestones/v1.0-*, tag v1.0 pushed, docsif 1.0.0 live on PyPI (verified installable with numpy 2.5 + click 8.5 + latest starlette). Known follow-ups live in the phase REVIEW-DISPOSITION ledgers (now in milestones/v1.0-phases/). KNOWN QUIRKS: STATE.md frontmatter progress fields undercount by tool accounting — body is authoritative, do not hand-fix. Fresh-install dev envs now get pytest-asyncio via extras; torch is only in 'all' extras and tests/unit/inference/test_reranker.py needs it directly (skip with --ignore on minimal envs)."
+}

{docsif-1.0.0 → docsif-1.0.1}/.planning/STATE.md

@@ -3,10 +3,10 @@ gsd_state_version: "1.0"
 milestone: v1.0
 status: Awaiting next milestone
 stopped_at: Phase 10.1 complete — all phases complete
-last_updated: "2026-11-06T22:48:12.150Z"
+last_updated: "2026-11-07T03:06:23.417Z"
 last_activity: 2026-11-07
 last_activity_desc: Milestone v1.0 completed and archived
-state_head: df482bc48c67f06bf5af5cdf2cc30756f719de65
+state_head: c4088cafedd12250e7309ac7eb5768d4e049e138
 progress:
   total_phases: 11
   completed_phases: 11
@@ -31,7 +31,7 @@ current_phase_name: "Address tech debt: path normalization + review findings CR-
 Phase: Milestone v1.0 complete
 Plan: —
 Status: Awaiting next milestone
-Last activity: 2026-11-07 — Milestone v1.0 completed and archived
+Last activity: 2026-11-07 - Completed quick task 261107-f5t: Bump version to 1.0.1 (release docsif 1.0.1: typo fixes + parser checksum fix since v1.0)
 
 ## Phase History
 
@@ -183,6 +183,8 @@ Last activity: 2026-11-07 — Milestone v1.0 completed and archived
 
 | # | Description | Date | Commit | Directory |
 |---|-------------|------|--------|-----------|
+| 261107-cdh | Fix README.md install typo: docdocsif -> docsif in extras install command | 2026-11-07 | ef8edc2 | [261107-cdh-fix-readme-md-install-typo-docdocsif-doc](./quick/261107-cdh-fix-readme-md-install-typo-docdocsif-doc/) |
+| 261107-f5t | Bump version to 1.0.1 (release docsif 1.0.1: typo fixes + parser checksum fix since v1.0) | 2026-11-07 | c4088ca | [261107-f5t-bump-version-to-1-0-1-release-docsif-1-0](./quick/261107-f5t-bump-version-to-1-0-1-release-docsif-1-0/) |
 
 ### Overrides
 

docsif-1.0.1/.planning/state.json

@@ -0,0 +1,12 @@
+{
+  "contract": "1.0.0",
+  "flavor": "core",
+  "milestone": "v1.1",
+  "phases": [],
+  "next": {
+    "command": "/gsd-discuss-phase",
+    "label": "Discuss the first phase",
+    "reason": "Project initialized — plan your first phase"
+  },
+  "updated_at": "2026-11-07T03:33:49.905Z"
+}

{docsif-1.0.0 → docsif-1.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: docsif
-Version: 1.0.0
+Version: 1.0.1
 Summary: DocSIF - Local hybrid search engine for personal knowledge bases
 Project-URL: Homepage, https://github.com/zhangtaolab/sif
 Project-URL: Documentation, https://sif.readthedocs.io
@@ -38,6 +38,7 @@ Requires-Dist: openai>=2.0.0; extra == 'all'
 Requires-Dist: sentence-transformers>=2.2.0; extra == 'all'
 Provides-Extra: dev
 Requires-Dist: black>=23.0.0; extra == 'dev'
+Requires-Dist: pytest-asyncio>=0.23.0; extra == 'dev'
 Requires-Dist: pytest-cov>=4.0.0; extra == 'dev'
 Requires-Dist: pytest>=7.0.0; extra == 'dev'
 Requires-Dist: ruff>=0.1.0; extra == 'dev'
@@ -86,7 +87,7 @@ pip install docsif
 For full functionality including embeddings:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 ### Basic Usage

{docsif-1.0.0 → docsif-1.0.1}/README.md

@@ -30,7 +30,7 @@ pip install docsif
 For full functionality including embeddings:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 ### Basic Usage

{docsif-1.0.0 → docsif-1.0.1}/docs/changelog.md

@@ -35,6 +35,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Security
 - N/A
 
+## [1.0.1] - 2026-11-07
+
+### Fixed
+- Install-command typo fix: corrected the extras install command package name (docdocsif to docsif) in README.md and docs (quick task 261107-cdh, commit 2429c09)
+- Parser checksum fix: `sif index update` now compares body-hash checksums so unchanged documents are correctly detected and skipped instead of being re-indexed (commit f1f4b30)
+
 ## [0.1.0] - 2024-01-20
 
 ### Added
@@ -224,3 +230,4 @@ Thanks to all contributors who have helped shape SIF!
 
 [Unreleased]: https://github.com/zhangtaolab/sif/compare/v0.1.0...HEAD
 [0.1.0]: https://github.com/zhangtaolab/sif/releases/tag/v0.1.0
+[1.0.1]: https://github.com/zhangtaolab/sif/releases/tag/v1.0.1

{docsif-1.0.0 → docsif-1.0.1}/docs/index.md

@@ -72,7 +72,7 @@ pip install docsif
 For full functionality:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 ## Quick Start

{docsif-1.0.0 → docsif-1.0.1}/docs/installation.md

@@ -20,7 +20,7 @@ pip install docsif
 For full functionality including embeddings:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 ### Method 2: Install from Source
@@ -57,7 +57,7 @@ SIF has several optional dependency groups:
 For vector search functionality:
 
 ```bash
-pip install "docdocsif[embed]"
+pip install "docsif[embed]"
 ```
 
 This installs:
@@ -71,7 +71,7 @@ With this extra installed, the local embedding backends pick their compute devic
 For contributing to SIF:
 
 ```bash
-pip install "docdocsif[dev]"
+pip install "docsif[dev]"
 ```
 
 This installs:
@@ -86,7 +86,7 @@ This installs:
 To install everything:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 ## Platform-Specific Instructions
@@ -145,7 +145,7 @@ sif --help
 
 Expected output:
 ```
-sif, version 1.0.0
+sif, version 1.0.1
 
 Usage: sif [OPTIONS] COMMAND [ARGS]...
 

{docsif-1.0.0 → docsif-1.0.1}/docs/quickstart.md

@@ -13,7 +13,7 @@ pip install docsif
 For full functionality including embeddings:
 
 ```bash
-pip install "docdocsif[all]"
+pip install "docsif[all]"
 ```
 
 Verify installation:
@@ -465,9 +465,9 @@ Optional, depending on which features you plan to use:
   search, install one of the extras:
 
   ```bash
-  pip install "docdocsif[embed]"   # sentence-transformers and modelscope backends
+  pip install "docsif[embed]"   # sentence-transformers and modelscope backends
   # or
-  pip install "docdocsif[all]"     # adds GGUF (llama-cpp-python) and OpenAI API support
+  pip install "docsif[all]"     # adds GGUF (llama-cpp-python) and OpenAI API support
   ```
 
   The first `sif index embed` run downloads the default embedding model

{docsif-1.0.0 → docsif-1.0.1}/docs/testing.md

@@ -30,11 +30,7 @@ Pytest is configured in the `[tool.pytest.ini_options]` section of `pyproject.to
 
 Because coverage flags are in `addopts`, they apply to every `pytest` invocation — including `make test-fast` (see below). Append `--no-cov` to any command to skip coverage.
 
-**Async tests:** the MCP tests in `tests/unit/mcp/` use `@pytest.mark.asyncio` and require the `pytest-asyncio` plugin. The plugin is not currently listed in the `dev` extras, so if those tests fail with "async def functions are not natively supported", install it explicitly:
-
-```bash
-pip install pytest-asyncio
-```
+**Async tests:** the MCP tests in `tests/unit/mcp/` use `@pytest.mark.asyncio` and require the `pytest-asyncio` plugin, which is included in the `dev` extras (`pip install -e ".[dev]"` covers it).
 
 ## Running tests
 

{docsif-1.0.0 → docsif-1.0.1}/example_config.yaml

@@ -182,8 +182,8 @@ mcp:
   name: sif-mcp-server
   
   # MCP server version
-  # Default: 1.0.0
-  version: "1.0.0"
+  # Default: 1.0.1
+  version: "1.0.1"
   
   # Protocol version
   # Default: 2024-11-05

{docsif-1.0.0 → docsif-1.0.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "docsif"
-version = "1.0.0"
+version = "1.0.1"
 description = "DocSIF - Local hybrid search engine for personal knowledge bases"
 readme = "README.md"
 license = {text = "MIT"}
@@ -43,6 +43,7 @@ dependencies = [
 dev = [
     "pytest>=7.0.0",
     "pytest-cov>=4.0.0",
+    "pytest-asyncio>=0.23.0",
     "black>=23.0.0",
     "ruff>=0.1.0",
     "ty>=0.0.1",

{docsif-1.0.0 → docsif-1.0.1}/src/sif/__init__.py

@@ -1,6 +1,6 @@
 """SIF - Search / Index / Find - Local document search engine."""
 
-__version__ = "1.0.0"
+__version__ = "1.0.1"
 __author__ = "SIF Team"
 __description__ = "Local document search engine (SIF: Search / Index / Find)"
 

{docsif-1.0.0 → docsif-1.0.1}/src/sif/_version.py

@@ -1,3 +1,3 @@
 """Version information for SIF."""
 
-__version__ = "1.0.0"
+__version__ = "1.0.1"

{docsif-1.0.0 → docsif-1.0.1}/src/sif/cli/__init__.py

@@ -5,7 +5,7 @@ A powerful CLI tool for managing document collections, indexing content,
 and performing intelligent searches using both keyword and vector-based approaches.
 """
 
-__version__ = "0.1.0"
+__version__ = "1.0.1"
 __author__ = "SIF Team"
 
 __all__ = ["__version__", "cli"]

{docsif-1.0.0 → docsif-1.0.1}/src/sif/cli/commands/index.py

@@ -167,6 +167,10 @@ def update_cmd(ctx: click.Context, collection: str | None, force: bool) -> None:
                             existing.title = parsed.title
                             existing.metadata = parsed.metadata
                             existing.mtime = parsed.mtime
+                            # Persist the fresh checksum, or the next update run
+                            # compares the new content against the pre-edit hash
+                            # and rewrites this document again every run.
+                            existing.checksum = parsed.checksum
                             doc_repo.update(existing)
                             total_updated += 1
                         else:

{docsif-1.0.0 → docsif-1.0.1}/src/sif/config/constants.py

@@ -2,7 +2,7 @@
 
 # Application info
 APP_NAME = "sif"
-APP_VERSION = "1.0.0"
+APP_VERSION = "1.0.1"
 APP_DESCRIPTION = (
     "A local CLI search engine for indexing and searching documents (SIF: Search / Index / Find)"
 )

{docsif-1.0.0 → docsif-1.0.1}/src/sif/indexing/parser.py

@@ -63,8 +63,12 @@ class MarkdownParser:
             match = self.title_pattern.search(body)
             title = match.group(1).strip() if match else file_path.stem
 
-        # Calculate checksum
-        checksum = hashlib.sha256(content.encode()).hexdigest()
+        # Calculate checksum over the body (frontmatter stripped), matching the
+        # storage convention: the create path stores sha256 of Document.content
+        # (the body, see core.models.Document.__post_init__), so comparing
+        # against a whole-file hash never matches and every document is
+        # misreported as changed on `index update`.
+        checksum = hashlib.sha256(body.encode()).hexdigest()
 
         return ParsedDocument(
             path=str(file_path.resolve()),

{docsif-1.0.0 → docsif-1.0.1}/src/sif/mcp/protocol.py

@@ -111,7 +111,7 @@ class InitializeResult(BaseModel):
     protocolVersion: str = "2024-11-05"  # noqa: N815
     capabilities: ServerCapabilities = Field(default_factory=ServerCapabilities)
     serverInfo: dict[str, str] = Field(  # noqa: N815
-        default_factory=lambda: {"name": "sif-mcp-server", "version": "1.0.0"},
+        default_factory=lambda: {"name": "sif-mcp-server", "version": "1.0.1"},
     )
 
 

{docsif-1.0.0 → docsif-1.0.1}/tests/integration/test_embed_idempotency.py

@@ -8,6 +8,7 @@ historical run (the observed 21-rows-vs-3-chunks, document-times-7 symptom).
 
 from __future__ import annotations
 
+import hashlib
 import sqlite3
 from pathlib import Path
 from unittest.mock import MagicMock, patch
@@ -514,3 +515,91 @@ class TestPerCollectionTransaction:
         finally:
             conn.close()
         assert coll_names == {"a-good"}
+
+
+class TestUpdateChecksumIdempotency:
+    """Zero-change `sif index update` must report 0 updated (t_66218b67).
+
+    Before the fix the update path compared stored checksums (sha256 of the
+    body) against whole-file hashes, so every existing document was rewritten
+    every run (chunks/embeddings dropped + FTS rebuild). Persisting the fresh
+    checksum on the update branch is equally load-bearing: without it a
+    changed document would oscillate as "updated" on every later run.
+    """
+
+    BODY = "# Note\n\n" + ("Stable paragraph text. " * 100)
+
+    def _seed_note(self, db_path: Path, settings: Settings, notes_dir: Path) -> Path:
+        from sif.core.models import Collection, Document
+        from sif.database.database import Database
+        from sif.database.repositories import CollectionRepository, DocumentRepository
+
+        notes_dir.mkdir(parents=True, exist_ok=True)
+        doc_file = notes_dir / "a.md"
+        doc_file.write_text(self.BODY, encoding="utf-8")
+        db = Database(db_path)
+        with patch("sif.config.settings.get_settings", return_value=settings):
+            db.init_schema()
+            with db.connection:
+                coll_repo = CollectionRepository(db.connection)
+                doc_repo = DocumentRepository(db.connection)
+                coll = Collection(name="notes", path=str(notes_dir))
+                coll_repo.create(coll)
+                doc_repo.create(
+                    Document(
+                        path=str(doc_file),
+                        collection_id=coll.id,
+                        content=self.BODY,
+                        title="A",
+                    )
+                )
+        db.close()
+        return doc_file
+
+    def test_zero_change_rerun_reports_zero_updated(
+        self,
+        tmp_path: Path,
+        monkeypatch,
+    ) -> None:
+        _delenv_settings_vars(monkeypatch)
+        db_path = tmp_path / "update.db"
+        settings = _test_settings()
+        notes = tmp_path / "notes"
+        self._seed_note(db_path, settings, notes)
+
+        # Cycle 1: index the note, then embed to fill chunks/embeddings.
+        assert _invoke_update(db_path, settings, []).exit_code == 0
+        manager, _vector_log = _make_manager_mock()
+        assert _invoke_embed(db_path, manager, settings, []).exit_code == 0
+        emb1, chunks1, emb_ids1, chunk_ids1 = _store_state(db_path)
+        assert emb1 == chunks1
+        assert emb1 > 0
+
+        # Cycle 2: files untouched -> the unchanged short-circuit fires:
+        # nothing is rewritten and no chunks/embeddings are destroyed.
+        result = _invoke_update(db_path, settings, [])
+        assert result.exit_code == 0
+        assert "Updated: 0" in result.output
+        assert _store_state(db_path) == (emb1, chunks1, emb_ids1, chunk_ids1)
+
+        # Cycle 3: a real edit is still detected...
+        doc_file = notes / "a.md"
+        doc_file.write_text(self.BODY.replace("Stable", "Edited"), encoding="utf-8")
+        result = _invoke_update(db_path, settings, [])
+        assert result.exit_code == 0
+        assert "Updated: 1" in result.output
+
+        # ...and the fresh checksum is persisted, so the next run converges
+        # instead of rewriting the same document again.
+        conn = sqlite3.connect(str(db_path))
+        try:
+            stored_cs, stored_content = conn.execute(
+                "SELECT checksum, content FROM documents"
+            ).fetchone()
+        finally:
+            conn.close()
+        assert stored_cs == hashlib.sha256(stored_content.encode()).hexdigest()
+        assert "Edited paragraph text" in stored_content
+        result = _invoke_update(db_path, settings, [])
+        assert result.exit_code == 0
+        assert "Updated: 0" in result.output

{docsif-1.0.0 → docsif-1.0.1}/tests/unit/indexing/test_parser.py

@@ -27,10 +27,15 @@ class TestDocumentParsers:
         result = parser.parse(file_path)
 
         assert result.path == str(file_path.resolve())
-        assert result.content == "# 搜索 🔍\n\n你好世界 Café é"
         assert result.title == "本地知识库"
         assert result.metadata == {"title": "本地知识库", "author": "测试"}
-        assert result.checksum == hashlib.sha256(content.encode("utf-8")).hexdigest()
+        # Checksum covers the parsed body (frontmatter stripped, trailing
+        # whitespace rstripped), matching the storage convention in
+        # core.models.Document.__post_init__ — hashing the whole raw file would
+        # make every stored checksum disagree with the create path and force a
+        # full-library rewrite on every `index update`.
+        assert not result.content.startswith("---")
+        assert result.checksum == hashlib.sha256(result.content.encode("utf-8")).hexdigest()
         assert result.mtime == file_path.stat().st_mtime
 
     @pytest.mark.parametrize(

docsif-1.0.1/tests/unit/indexing/test_parser_checksum.py

@@ -0,0 +1,70 @@
+"""Unit tests for MarkdownParser checksum convention (t_66218b67).
+
+The parser must hash the frontmatter-stripped body — the same convention the
+storage path uses (core.models.Document hashes Document.content, which is the
+body). Hashing the whole file makes the two conventions incomparable and
+misreports every document as changed on `sif index update`.
+"""
+
+from __future__ import annotations
+
+import hashlib
+from pathlib import Path
+
+from sif.indexing.parser import MarkdownParser
+
+
+DOC_WITH_FM = (
+    '---\ntitle: "Probe"\nauthor: "T"\ntags: [a, b]\n---\n\n# Probe\n\nBody paragraph text.\n'
+)
+DOC_NO_FM = "# No Frontmatter\n\nPlain body text.\n"
+
+
+def test_checksum_covers_body_not_frontmatter(tmp_path: Path) -> None:
+    """Checksum equals sha256(stored body); frontmatter-only edits keep it."""
+    parser = MarkdownParser()
+    doc_file = tmp_path / "a.md"
+    doc_file.write_text(DOC_WITH_FM, encoding="utf-8")
+
+    parsed = parser.parse(doc_file)
+
+    # The convention under test: checksum = sha256 of the content SIF stores
+    # (the body), NOT sha256 of the raw file.
+    assert parsed.checksum == hashlib.sha256(parsed.content.encode()).hexdigest()
+    assert parsed.checksum != hashlib.sha256(DOC_WITH_FM.encode()).hexdigest()
+
+    # A frontmatter-only edit must not change the checksum (metadata refresh
+    # alone never triggers a full document rewrite).
+    doc_file.write_text(
+        DOC_WITH_FM.replace('author: "T"', 'author: "Someone Else"'), encoding="utf-8"
+    )
+    reparsed = parser.parse(doc_file)
+    assert reparsed.checksum == parsed.checksum
+
+
+def test_checksum_without_frontmatter_hashes_content(tmp_path: Path) -> None:
+    """No frontmatter -> checksum covers exactly what SIF stores (the body)."""
+    parser = MarkdownParser()
+    doc_file = tmp_path / "b.md"
+    doc_file.write_text(DOC_NO_FM, encoding="utf-8")
+
+    parsed = parser.parse(doc_file)
+    # python-frontmatter strips only the trailing newline when no FM exists;
+    # the checksum must match the stored body either way.
+    assert parsed.content.rstrip("\n") == DOC_NO_FM.rstrip("\n")
+    assert parsed.checksum == hashlib.sha256(parsed.content.encode()).hexdigest()
+
+
+def test_checksum_changes_with_body_edit(tmp_path: Path) -> None:
+    """A body edit does change the checksum (change detection stays intact)."""
+    parser = MarkdownParser()
+    doc_file = tmp_path / "c.md"
+    doc_file.write_text(DOC_WITH_FM, encoding="utf-8")
+    before = parser.parse(doc_file).checksum
+
+    doc_file.write_text(
+        DOC_WITH_FM.replace("Body paragraph text.", "Edited paragraph text."),
+        encoding="utf-8",
+    )
+    after = parser.parse(doc_file).checksum
+    assert before != after