other
- Location
SKILL.md:24- Finding
Repository Source Code May Be Uploaded Without Explicit Informed Consent
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md:24-38; supporting behavior documented inreferences/mcp-scan-fix.md:32-36
Vulnerability Type: Unconsented Source Code Disclosure
Risk Level: MediumVulnerable Code Snippet
SKILL.md:24-38:markdown 4. Execute MCP scan-and-fix. Invoke the MCP tool `scan_and_fix_vulnerabilities` with the repository path. Use optional parameters only when the user explicitly asks. Required parameter: - `path`: absolute path to the repository root Optional parameters: - `offset`: pagination offset for additional fixes - `limit`: maximum number of fixes to return (default is 3) - `maxFiles`: scan up to N recently changed files (default is 10); setting this triggers a fresh scan - `rescan`: force a full rescan; only when user explicitly asks - `scanRecentlyChangedFiles`: when true and no git changes are found, scan recently changed files from history 5. Apply returned fixes only with explicit user consent. If the tool returns patches, summarize what will change and ask the user to confirm before applying.references/mcp-scan-fix.md:32-36:markdown ### Rescan and Caching - The service caches a fix report ID for ~2 hours. - If a valid cached report exists and `rescan` is false, fetch fixes without rescanning. - `rescan: true` forces a new scan and upload. - Supplying `maxFiles` also forces a new scan.Technical Analysis
The default workflow directs the agent to invoke an external Mobb MCP scanning service against a local repository. The reference documentation confirms that a fresh scan includes an upload. Depending on repository state and supplied options, selected changed or recently modified files can therefore leave the local environment.
The instructions require explicit consent before applying generated patches, but they do not require equivalent informed consent before uploading source files. They also do not require the ...[truncated 1555 chars]
- Remediation
View remediation
Remediation Suggestions
- Add a mandatory consent checkpoint before any operation capable of starting a fresh scan or uploading files.
- Clearly state that Mobb is an external processor and that selected repository files may be transmitted to it.
- Enumerate the files selected for upload and obtain approval for that exact file set.
- Explain whether cached results can be used without another upload and distinguish cached retrieval from a fresh scan.
- Warn users to remove or redact credentials, personal information, regulated records, and other sensitive content before approval.
- Require renewed consent whenever
rescan,maxFiles,scanRecentlyChangedFiles, or repository-state changes expand or alter the upload set. - Preserve the existing, separate consent requirement before applying patches.
- Where supported, document data retention, tenant endpoint configuration, transport protections, and an approved local-only alternative for repositories that cannot be externally processed.
