Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
slackdump mcpcan now start without an archive path. If no<archive>argument is given, the server starts with no active source and logs an advisory; an agent can then callload_sourceto open one at runtime.load_sourcetool: new MCP tool that lets a connected agent open (or switch) the active Slackdump archive without restarting the server. Only one source is held at a time; the previous one is closed before the new one is opened.Server.srcis guarded by async.RWMutex— read-lock for all data handlers, write-lock forloadSource().Servernow usesOption/WithSource()/WithLogger()/WithSourceLoader()— no required constructor arguments.source.SourceResumeCloser: composite interface (Sourcer + Resumer + io.Closer) used as the type forServer.src;MockSourceResumeClosergenerated insource/mock_source.cmd/slackdump/internal/mcp/assets/mcp.mdfully rewritten (optional arg, all tool parameters documented, integration examples for stdio/HTTP/load_source, corrected listen address127.0.0.1:8483).copilot-instructions.mdoverhauled: correct Go version, module pathv4, full package tree, MCP server section, command help-page convention.PKG=.../v3→v4.load_sourcehandler and nil-source guards on all data handlers.