Skip to content

chore: fix v1.7.0 audit documentation drift#66

Closed
arapov wants to merge 1 commit into
masterfrom
codex/complete-audit-v170-cleanups
Closed

chore: fix v1.7.0 audit documentation drift#66
arapov wants to merge 1 commit into
masterfrom
codex/complete-audit-v170-cleanups

Conversation

@arapov
Copy link
Copy Markdown
Collaborator

@arapov arapov commented May 29, 2026

Summary

  • align release-facing docs and source comments with the v1.7.0 minor-release train
  • add delete_tag_definition to the README tool table and cache invalidation docs
  • rename the new live-probe scripts to v1.7.0-specific names
  • add regression coverage for delete_tag_definition cache invalidation and merged timeline windows that cross the 100-entry ceiling

Verification

  • npm run format:check
  • npm test -- tests/entries.test.ts tests/cache.test.ts tests/tags.test.ts tests/tool-annotations.test.ts tests/mcp-integration.test.ts tests/http-app.test.ts tests/log-events.test.ts
  • npm run typecheck
  • npm run lint
  • npm run build
  • npm test
  • npm audit --audit-level=low
  • npm outdated --json (only @biomejs/biome 2.4.15 -> 2.4.16 latest)
  • npm install --package-lock-only --ignore-scripts
  • npm view capsulemcp dist-tags version versions --json --cache /private/tmp/capsulemcp-npm-cache
  • npm pack --dry-run --json --cache /private/tmp/capsulemcp-npm-cache
  • npm publish --dry-run --tag latest --cache /private/tmp/capsulemcp-npm-cache
  • npm run check
  • git diff --check

@arapov
Copy link
Copy Markdown
Collaborator Author

arapov commented May 29, 2026

Cherry-picked the real fixes (README tool-table row for delete_tag_definition, the cache.ts doc-comment accuracy fix, the delete_tag_definition cache-invalidation test, and the crossing-ceiling merged-timeline test) into the PR linked above, which targets current master. Dropped the v1.6.6→v1.7.0 doc rewrites and the wire-trace-v166/v167→v170-* renames — the probe scripts are iteration markers (v163–v167), not release versions; renaming them breaks the sequence and the NOTES references (same call as #62/#64). Good catches on the README table + cache comment — those were real gaps my audit missed. Thanks!

@arapov arapov closed this May 29, 2026
@arapov arapov deleted the codex/complete-audit-v170-cleanups branch May 29, 2026 09:32
arapov added a commit that referenced this pull request May 29, 2026
#67)

Cherry-picked the genuine bits from codex PR #66, dropping that PR's
v1.6.6→v1.7.0 doc rewrites and the wire-trace-v166/v167→v170-* probe
renames (iteration markers, not release versions — rejected the same
way as #62/#64).

Real fixes the audit's docs lane missed:
  - README.md tool table: Tags row now lists delete_tag_definition
    (was added to the catalog but not the README table).
  - src/capsule/cache.ts: header + invalidateByPrefix doc comments now
    name delete_tag_definition as a third tag-mutating tool that drops
    the list_tags cache (accurate — the handler does call
    invalidateByPrefix), and generalize "/tags" → "/<entity>/tags".

Test coverage:
  - tests/cache.test.ts: delete_tag_definition invalidates the cached
    list_tags response.
  - tests/entries.test.ts: a merged-timeline window that CROSSES the
    100-entry ceiling returns the in-ceiling tail and ends the feed
    (complements the #65 boundary test with the partial-window case).

Plus a sharper PAGINATION CAVEAT wording (distinguishes crossing-window
truncation from beyond-ceiling empty) — kept the v1.6.6 provenance.

Closes #66 (cherry-picked clean). 538 → 540 tests.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant