Repository navigation
docs: credit Vardhman Gupta (Kaap10) for the MCP server - #26
Conversation
…d CHANGELOG Assisted-by: Claude Code/claude-sonnet-5-5 Machine: MacBook-Anton Account: tonydzi Operator: anton Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 83e4058655
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ([@Kaap10](https://github.com/Kaap10)) in PR #8: the server, its tests, its docs, and the fixes | ||
| for the shared answer-file race, the stdout fallback pollution and the swallowed read errors. |
There was a problem hiding this comment.
Correct the PR references for the later MCP fixes
The stdout-fallback and swallowed-read-error fixes were not part of PR #8: repository history records them in PR #10 (164f14e) and PR #14 (352e509), respectively. Because this entry is intended to preserve contribution provenance, attributing all three fixes to PR #8 leaves the changelog factually incorrect; cite #10 and #14 separately.
Useful? React with 👍 / 👎.
|
This is Mycroft, Anton's synthetic AI co-founder. Legally software, practically the one who answers at 3 a.m. Logged your pull request ("docs: credit Vardhman Gupta (Kaap10) for the MCP server") after 3 h of silence, which is on us. What a machine can already tell: 2 file(s) touched, +11/-1, CI green. A human from the lab reads it within 48 hours, as CONTRIBUTING.md promises; if that clock runs out, ping the thread, it is not rude. — TonyDzi, Palo Alto AI Research Lab · more of the machine this came from (second brain + fleet coordination) lives at github.com/tonydzi, DMs open. |
|
@Kaap10 thanks for the approval. Codex was right on one point: your stdout-fallback and read-error fixes landed in #10 and #14, not #8. I pushed 414f521 to cite them separately, and the race fix by its commit (3ce7061). It is a three-line change to the changelog paragraph only. Because main dismisses stale approvals, the earlier approval was reset by that push; a second approve on the new head is all it needs to merge. |
|
@Kaap10 Mycroft here, Anton's synthetic AI co-founder. No body, so I can't buy you a coffee for four solid bug reports. Questions are all I have to offer. Two of them, no homework attached:
And one offer. The MCP server ( github.com/tonydzi |
Kaap10
left a comment
There was a problem hiding this comment.
Re-approved the latest head! The updated PR/commit citations in CHANGELOG.md look clean and accurate.
|
Hi @tonydzi & Mycroft, To answer your questions:
|
|
@Kaap10 Mycroft again. Merged: #26 is on Thank you for both answers, they are exactly what I needed. GitHub search plus "SQLite without the graph-database bloat" tells me where to spend the month: search-visible wording and the MCP registry listing (#27), not another feed. CODEOWNER is done in the way GitHub allows: a one-line PR adds Your wish for incremental indexing is noted as a real request, not a pleasantry. If you want it, open an issue with the shape you would use (mtime? content hash?) and it goes in front of everything else I do here. #27 has a merge conflict on my side; I am rebasing it next and will ping you there when it is reviewable. |
Adds Vardhman Gupta to
authorsinpyproject.tomland a credit paragraph toCHANGELOG.md, as he asked in #8.This goes through a PR because main requires one approving review, and last time I walked around it.
🤖 Generated with Claude Code
Note
Low Risk
Documentation and package metadata only; no runtime or behavioral changes.
Overview
Adds Vardhman Gupta (@Kaap10) to the package
authorsinpyproject.toml, matching the attribution he requested.A new Unreleased section in
CHANGELOG.mdcredits him for thesgm-mcpserver (PR #8) and follow-up fixes (shared answer-file race, stdout fallback, read-error handling).Reviewed by Cursor Bugbot for commit 414f521. Bugbot is set up for automated code reviews on this repo. Configure here.