refactor(adapters): remove dead vitest setup from hermes-gateway shim (#18281)

## Thinking Path

> - Paperclip is the open source app people use to manage AI agents for
work.
> - The `packages/adapters/hermes-gateway` package is a deprecated
compatibility shim. It forwards to
`@paperclipai/hermes-paperclip-adapter/gateway`.
> - The shim carries a `vitest.config.ts` file and a `test` script. The
package contains no test files. It has contained none since it was
created in #8543.
> - The root Vitest project list does not select this package. No
workflow and no repository command runs its `test` script. The script
starts Vitest with no test files, so it can only fail.
> - This pull request deletes the dead Vitest config file and the broken
`test` script line. The package code, its exports, and its
release-manifest entry stay unchanged.
> - The benefit is less dead test support and one less broken entry
point in a deprecated package.

## Linked Issues or Issue Description

**Current behavior**

The deprecated `@paperclipai/adapter-hermes-gateway` shim has a
`vitest.config.ts` file and a `test` script. The package has no test
files. No CI lane and no repository command selects the package for
tests. The script cannot pass, because Vitest exits with an error when
it finds no test files.

**Proposed behavior**

Delete the unused Vitest config file. Remove the `test` script from the
package manifest. Keep the published code and the release-manifest entry
unchanged.

**Reason and benefit**

This is dead test support. It protects no contract, because no test
exists in the package. The `test` script misleads a reader: it looks
like a test entry point, but it always fails. The real Hermes Gateway
coverage lives in `packages/adapters/hermes`, which the default test run
selects. This change came out of a test-suite maintenance audit. (Refs
#8543, which added the shim with this config.)

**Breaking changes**

None. No command, workflow, or package invokes the removed script or
config. The published package code does not change. The `vitest` dev
dependency stays, because pull requests may not edit the lockfile.

## What Changed

- Deleted `packages/adapters/hermes-gateway/vitest.config.ts`. The
package has no test files, so the config selects nothing.
- Removed the `test` script from
`packages/adapters/hermes-gateway/package.json`. The script ran `vitest
run` with no test files and could only fail.

## Verification

- Searched the repository for references to the deleted config and the
removed script. The search `rg
'hermes-gateway/vitest|vitest.*hermes-gateway'` returns no match outside
the lockfile. No workflow, script, or document names them.
- Confirmed that the root Vitest project list (`vitest.config.ts`) and
the shard runner (`scripts/run-vitest-stable.mjs`) do not select this
package.
- Confirmed that no open pull request changes these two files. The check
read the file list of every open pull request.
- Ran locally, without an install: `node --test
scripts/release-package-map.test.mjs`. It passes. It asserts the shim's
release-manifest entry. `node scripts/release-package-map.mjs check`
also passes: "30 enabled for CI publish, 5 disabled pending bootstrap".
- Parsed `packages/adapters/hermes-gateway/package.json` with Node. The
JSON is valid.
- Not run locally: `pnpm -r typecheck`, `pnpm test:run`, `pnpm build`.
The execution workspace has no installed dependencies. The pull request
CI runs these lanes against this change.

## Risks

- Low risk. The deleted config file is outside the package
`tsconfig.json` include list (`["src"]`), so typecheck output does not
change.
- If a future change adds tests to this package, it must add a test
script again. The removed config held only default settings, so a new
one is trivial to write.
- The `vitest` dev dependency remains. Removing it would change
`pnpm-lock.yaml`, which pull requests may not edit.

> For core feature work, check [`ROADMAP.md`](ROADMAP.md) first and
discuss it in `#dev` before opening the PR. Feature PRs that overlap
with planned core work may need to be redirected — check the roadmap
first. See [`CONTRIBUTING.md`](CONTRIBUTING.md).

## Model Used

- Z.ai GLM-5.3 served through OpenRouter. Exact model ID: `z-ai/glm-5.3`
(route `openrouter/z-ai/glm-5.3`).
- Standard agent mode with tool use: file search, file edit, shell, and
GitHub CLI in a sandboxed Linux workspace.
- Context window size: not independently verified. The change did not
rely on a large context.

## Checklist

- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used (with version and capability
details)
- [x] I have checked ROADMAP.md and confirmed this PR does not duplicate
planned core work
- [x] I have searched GitHub for duplicate or related PRs and linked
them above
- [x] I have either (a) linked existing issues with `Fixes: #` /
`Closes: #` / `Refs: #` OR (b) described the issue in-PR following the
relevant issue template
- [x] I have not referenced internal/instance-local Paperclip issues or
links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip`
URLs)
- [x] My branch name describes the change (e.g. `docs/...`, `fix/...`)
and contains no internal Paperclip ticket id or instance-derived details
- [x] I have run tests locally and they pass (release-package-map tests;
broad suites run in PR CI — see Verification)
- [x] I have added or updated tests where applicable (not applicable:
test-support deletion only; the retained release-manifest test still
covers the manifest contract)
- [x] I have updated relevant documentation to reflect my changes (not
applicable: no document references the removed script or config)
- [x] I have considered and documented any risks above
- [ ] All Paperclip CI gates are green
- [ ] Greptile is 5/5 with no open P2s, recommendations, or follow-ups
- [x] I will address all Greptile and reviewer comments before
requesting merge
This commit is contained in:
Dotta authored and GitHub committed 2026-10-30 09:09:29 -05:00
1 parent 5a7a6b32ba
commit 65554ee596
2 files changed
+1 -10

No files matched your search

@@ -64,8 +64,7 @@
"scripts": {
"build": "tsc",
"clean": "rm -rf dist",
"typecheck": "tsc --noEmit",
"test": "vitest run"
"typecheck": "tsc --noEmit"
},
"dependencies": {
"@paperclipai/hermes-paperclip-adapter": "workspace:*"
@@ -1,8 +0,0 @@
import { defineConfig } from "vitest/config";
export default defineConfig({
test: {
environment: "node",
exclude: ["dist/**", "node_modules/**"],
},
});