Skip to content

feat(mcp): timestamp search, delete-by-query, slash-command docs; chore: untrack local build/venv dirs - #97

Merged
jose-compu merged 5 commits into
mainfrom
release/0.7.8-mcp-issues-94-96
May 11, 2026
Merged

feat(mcp): timestamp search, delete-by-query, slash-command docs; chore: untrack local build/venv dirs#97
jose-compu merged 5 commits into
mainfrom
release/0.7.8-mcp-issues-94-96

Conversation

@jose-compu

Copy link
Copy Markdown
Owner

Summary

Ships 0.7.8 MCP improvements: optional ISO timestamp bounds on logosdb_search (wired to Node searchTsRange), semantic delete via logosdb_delete with query / search_top_k / match_rank, and root README + .claude/commands docs for Claude Code slash flows. Stops tracking accidentally committed build-san/, build-fuzz/, and .venv-test/ and ignores them going forward.

Closes #94, closes #95, closes #96

Changes

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring (no functional changes)

Testing

  • Unit tests added/updated
  • Integration tests pass
  • Manual testing performed

Describe the tests you ran:

  • mcp/: npm install --ignore-scripts, npm run build (tsc)
  • (Recommended before merge): ctest --output-on-failure, pytest from a clean build/ as in CI

Checklist

  • Code compiles without warnings on GCC and Clang
  • Tests pass (ctest --output-on-failure, pytest)
  • Benchmarks run if performance-related
  • CHANGELOG updated for user-facing changes
  • Documentation updated (README, headers, guides)
  • PR title follows conventional commit style (feat:, fix:, docs:, etc.)

Additional Notes

  • logosdb_delete: callers must supply either id or query, not both. By-id delete still requires the namespace handle to be open (same as before); query-based delete can open an existing on-disk namespace using the embedding dimension from the query vector.
  • logosdb_search: timestamp filtering is active when ts_from and/or ts_to is non-empty; otherwise behavior matches the previous plain semantic search.
  • Repo hygiene: git rm --cached only untracks paths; contributors may still have local build-san, build-fuzz, .venv-test folders until they delete them manually if desired.
  • If this PR does not include the gitignore / index cleanup, drop those bullets and narrow the title to the MCP/docs/version slice only.

@jose-compu jose-compu self-assigned this May 11, 2026
@jose-compu jose-compu added documentation Improvements or additions to documentation portability Cross-platform support labels May 11, 2026
@jose-compu

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits May 11, 2026 06:39
Resolve conflicts keeping 0.7.8 version across all packages. Incorporate
security hardening from 0.7.7 (clampTopK, security.ts) while preserving
0.7.8 features: ts-range search (clampTopK now applied), delete-by-query,
and slash-command docs.

Co-authored-by: jose-compu <30682875+jose-compu@users.noreply.github.com>
…ve merge conflicts

Agent-Logs-Url: https://github.com/jose-compu/logosdb/sessions/6dabbd9b-a41a-4c2a-95b4-86e31344a401

Co-authored-by: jose-compu <30682875+jose-compu@users.noreply.github.com>

Copilot AI commented May 11, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merged main (0.7.7 security hardening) into this branch and resolved all conflicts in commit 452e63a:

  • Version files (CMakeLists.txt, logosdb.h, package.json × 3, pyproject.toml, package-lock.json): kept 0.7.8
  • CHANGELOG: 0.7.8 entry first, followed by the 0.7.7 entry from main
  • mcp/src/index.ts: applied clampTopK from the 0.7.7 security import while keeping the 0.7.8 timestamp-range search logic intact
  • New files from main: mcp/src/security.ts, mcp/src/security.test.ts, mcp/.npmignore, tools/fuzz_jsonl_metadata.cpp

Also added permissions: contents: read to the fuzz-smoke-jsonl CI job (CodeQL finding, b133e44). TypeScript builds cleanly (tsc exits 0).

@jose-compu
jose-compu merged commit bf2647c into main May 11, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation portability Cross-platform support

Projects

None yet

2 participants