Skip to content

fix: separate docs pointer from sync pointer to fix update --docs drift#849

Open
jyotirmya17 wants to merge 1 commit into
repowise-dev:mainfrom
jyotirmya17:fix/last-docs-commit-847
Open

fix: separate docs pointer from sync pointer to fix update --docs drift#849
jyotirmya17 wants to merge 1 commit into
repowise-dev:mainfrom
jyotirmya17:fix/last-docs-commit-847

Conversation

@jyotirmya17

@jyotirmya17 jyotirmya17 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Fixes #847.

What changed: Splits last_sync_commit and last_docs_commit to resolve pointer drift when running the update command with --index only.

How to test/verify:

Run an update with --index only.
Verify that last_sync_commit stays the same while last_docs_commit advances.
Unit tests have been added/updated in test_issue_847_fix.py to cover this behavior.

@repowise-bot

repowise-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

✅ Health: 7.6 (unchanged)

📋 At a glance
2 hotspots touched · 4 new findings introduced · 5 co-change pairs left out · 2 files with recent fix history. Scoped to packages.

⚠️ Change risk: moderate (riskier than 43% of this repo's commits · raw 8.6/10)
This change's risk is driven by:

  • more lines added than baseline

🩹 Review priority (files here with the most recent bug-fix history — defects cluster, so review these first)

🔎 More signals (2)

🔥 Hotspots touched (2)

  • .../update_cmd/command.py — 18 commits/90d, 3 dependents · primary owner: Raghav Chamadiya (91%)
  • .../update_cmd/persistence.py — 11 commits/90d, 3 dependents · primary owner: Raghav Chamadiya (100%)

🔗 Hidden coupling (2 files)

  • .../update_cmd/command.py co-changes with these files (not in this PR):
    • docs/CLI_REFERENCE.md (6× — 🟢 routine)
    • .../workspace/update.py (4× — 🟢 routine)
    • .../cli/helpers.py (4× — 🟢 routine)
  • .../update_cmd/persistence.py co-changes with these files (not in this PR):
    • .../pipeline/persist.py (5× — 🟢 routine)
    • .../pipeline/incremental.py (4× — 🟢 routine)

📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-15 14:43 UTC
Silence on a single PR with [skip repowise] in the title · Per-repo toggle on repowise.dev/settings?tab=bot

@jyotirmya17

Copy link
Copy Markdown
Contributor Author

I've also spun off the 3 related items discussed in #847 into their own issues here: #850, #851, and #852!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] index-only update advances last_sync_commit, making stale prose unreachable by update --docs

1 participant