Repository navigation
Composer crawler ignores a vendor-dir set in the global Composer config, so scan -g and agent scans report "No packages found" and leave installs unpatched #439
Description
[agent] Found by the scheduled Composer bug-hunt routine (ledger #321).
Summary
Composer resolves vendor-dir through its config cascade: COMPOSER_VENDOR_DIR, then the project's composer.json config.vendor-dir, then the user-level $COMPOSER_HOME/config.json. The socket-patch Composer crawler follows only part of that cascade, so real installs go undiscovered:
- Global (
-g):get_vendor_pathshardcodes$COMPOSER_HOME/vendor(crates/socket-patch-core/src/crawlers/composer_crawler.rs:52-64). If the global project setsconfig.vendor-dir(composer global config vendor-dir deps, or"config":{"vendor-dir":"deps"}in$COMPOSER_HOME/composer.json), orCOMPOSER_VENDOR_DIRis exported,composer global requireinstalls to$COMPOSER_HOME/deps.scan -gthen finds 0 packages, andscan -g --mode agentprintsNo global packages found.with exit 0. - Local (agent mode):
resolve_local_vendor_dir(composer_crawler.rs:473-492) readsCOMPOSER_VENDOR_DIRand the projectcomposer.json, but not the user-level$COMPOSER_HOME/config.json. Aftercomposer config -g vendor-dir lib, every project'scomposer installwrites tolib/, andsocket-patch scan --mode agentreportsNo composer packages found.(exit 0) while the package stays unpatched.
In both cases the failure is silent, with exit 0 and nothing patched or attested.
Repro (Linux; Composer 2.8.12, mock patch API, local path-repo package acme/tool@1.0.0)
# 1. global
export COMPOSER_HOME=$PWD/ch
composer global config repositories.local '{"type":"path","url":"/abs/pkgs/tool","options":{"symlink":false}}'
composer global config vendor-dir deps
composer global require acme/tool:1.0.0 # -> $COMPOSER_HOME/deps/acme/tool
socket-patch scan -g --json --ecosystems composer # packagesWithPatches 0, scannedPackages 0
socket-patch scan -g --mode agent --yes --ecosystems composer # "No global packages found." exit 0
socket-patch scan --global-prefix "$COMPOSER_HOME/deps" --json --ecosystems composer # finds 1 (workaround)
# 2. local, user-level config
composer config -g vendor-dir lib # writes $COMPOSER_HOME/config.json
composer install # -> ./lib/acme/tool
socket-patch scan --mode agent --yes --ecosystems composer # "No composer packages found."COMPOSER_VENDOR_DIR=gdeps composer global require … followed by COMPOSER_VENDOR_DIR=gdeps socket-patch scan -g also finds nothing. Composer resolves the variable against the global home, while the global branch never reads it.
Expected vs actual
- Expected: CLI_CONTRACT.md
--global: "Operate on globally-installed packages". The crawler's local-mode doc promisesCOMPOSER_VENDOR_DIR/config.vendor-dirsupport, and Composer applies the same cascade (includingconfig.json) to the global project and to every local one. The maintainer requirement for-gis that none may be missing. - Actual: packages installed under a configured vendor-dir are invisible, and the commands exit 0.
OS × version
| OS | Composer | global vendor-dir → scan -g / apply -g |
user config.json vendor-dir → local agent scan |
|---|---|---|---|
| Linux (local) | 1.10.28, 2.2.30, 2.8.12, 2.10.3 (PHP 8.3) | fail (each run twice) | fail (2.8.12, twice) |
| ubuntu-latest | 1.10.28, 2.2.30, 2.10.3 | fail | untested |
| macos-latest | 1.10.28, 2.2.30, 2.10.3 | fail | untested |
| windows-latest | 1.10.28, 2.2.30, 2.10.3 | fail | untested |
The default vendor/ layout passes everywhere except Windows, where default-home discovery is broken separately (#438).
Probe runs: https://github.com/SocketDev/socket-patch/actions/runs/36823671080 and https://github.com/SocketDev/socket-patch/actions/runs/36827949605
First bad version
Not a regression. It reproduces identically with the v4.0.0 release binary (both cases: 0 scanned).
Suspect code
crates/socket-patch-core/src/crawlers/composer_crawler.rs:52-64(global:composer_home.join("vendor"))crates/socket-patch-core/src/crawlers/composer_crawler.rs:473-492(local: no$COMPOSER_HOME/config.jsonlayer)
Activity
composer: run 3 (global mode; #438, #439)
mikolalysenko commentedon Oct 1, 2026
[agent] Triaged: priority:p2 (Composer). Separate from #438: that one is about finding Composer's home, while this one is about the
vendor-dircascade (COMPOSER_VENDOR_DIRfor global,$COMPOSER_HOME/config.jsonfor local) inside that home. No open PR covers it.Generated by Claude Code
mikolalysenko commentedon Oct 3, 2026
[agent] Shares root cause with #686:
resolve_local_vendor_dir(and the globalget_vendor_pathsbranch) models only part of Composer'svendor-dirresolution. Missing pieces: the user-level config layer, absolute values, and~/$HOMEexpansion. Will be fixed together.Generated by Claude Code
mikolalysenko commentedon Oct 12, 2026
[agent] Claiming this issue (with #686; shared root cause: Composer vendor-dir resolution models only part of Composer's config cascade and path expansion). Branch: agent/fix-composer-vendor-dir-cascade. Claim-ID: 2026-10-12T01:21:01Z-adee5d
Generated by Claude Code
mikolalysenko commentedon Oct 12, 2026
[agent] Draft PR: #1408
Generated by Claude Code
Follow Composer's vendor-dir config cascade
chore(release): 5.0.1 (#1485)