Skip to content

test(atlas): tripwire the bundled @synsci/atlas dependency against drift#115

Merged
Aayam Bansal (aayambansal) merged 1 commit into
mainfrom
test/atlas-dependency-tripwire
Jul 6, 2026
Merged

test(atlas): tripwire the bundled @synsci/atlas dependency against drift#115
Aayam Bansal (aayambansal) merged 1 commit into
mainfrom
test/atlas-dependency-tripwire

Conversation

@aayambansal

Copy link
Copy Markdown
Member

The agent shells out to the bundled atlas CLI to populate the graph (atlas doctor, nodes:create, evidence:add) — so even when openscience project init creates the root, a wrong/missing atlas binary leaves it bare. node_modules had silently drifted to 0.5.12 while package.json pins ^0.13.2, meaning the agent could run a different CLI surface than we ship.

Adds a tripwire (mirroring the model-pin delisting test from #91) that asserts:

  1. @synsci/atlas is declared with a concrete version range.
  2. The installed version satisfies that range (Bun.semver.satisfies) — catches drift.
  3. The package exposes its atlas bin entry.

Since @synsci/atlas is an optionalDependency (absent in the standalone-compiled build with no node_modules), tests 2–3 skip gracefully when it isn't installed. In CI the frozen-lockfile install resolves 0.13.2, so all three pass.

The agent shells out to the bundled `atlas` CLI to populate the graph
(atlas doctor, nodes:create, evidence:add). node_modules had drifted to
0.5.12 while package.json pins ^0.13.2 — a silent skew that runs a
different CLI surface than we ship, degrading graph population even when
`openscience project init` succeeds. Add a tripwire (mirroring the model-pin
delisting test) asserting the installed atlas satisfies the declared range
and exposes its `atlas` bin. Gracefully skips when atlas isn't installed
(a standalone-compiled build has no node_modules).
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openscience Ready Ready Preview, Comment Jul 6, 2026 4:00pm

Request Review

@aayambansal Aayam Bansal (aayambansal) merged commit 63564fd into main Jul 6, 2026
11 checks passed
@aayambansal Aayam Bansal (aayambansal) deleted the test/atlas-dependency-tripwire branch July 6, 2026 16:01
Aayam Bansal (aayambansal) added a commit that referenced this pull request Jul 8, 2026
…ift (#115)

The agent shells out to the bundled `atlas` CLI to populate the graph
(atlas doctor, nodes:create, evidence:add). node_modules had drifted to
0.5.12 while package.json pins ^0.13.2 — a silent skew that runs a
different CLI surface than we ship, degrading graph population even when
`openscience project init` succeeds. Add a tripwire (mirroring the model-pin
delisting test) asserting the installed atlas satisfies the declared range
and exposes its `atlas` bin. Gracefully skips when atlas isn't installed
(a standalone-compiled build has no node_modules).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant