Skip to content

Delete tests retired by the chat and catalog cuts - #809

Merged
TheGreatAxios merged 1 commit into
mainfrom
stale-tests
Sep 17, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
stale-tests

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Deletes workbench-host test cases in packages/agent-directory (route 404 guards, slug resolution, status, skill-pin) now that workbench-host filtering by name is gone — hostGuardedRow only checks for a missing asset, and a workbench is a plain tenant.
  • Deletes the purposeAgentDefinitions catalog-utility test in packages/agent-directory/src/client.test.ts, which only asserted the names of workflows (workbench-digest, last-30-days-research) removed from the trimmed workflow catalog.
  • Deletes packages/workflows/test/unreachable-message-contract.test.ts: the pinned vendor throw string no longer appears in vendor/intx/hub-sessions/src/session-service.ts at all (the vendor file changed); isAgentUnreachableError is still used elsewhere (agent-directory, chat, workflows) so the classifier itself stays.
  • Fixes packages/webhook-triggers/test/launch.test.ts's fake crypto provider, which returned the raw signed input instead of a 64-byte Ed25519 signature (breaking every test that reaches the sign step), and corrects one assertion's expected routeMail call count to match deliverWhenRoutable's existing one-retry-when-routable contract instead of assuming no retry.

Test plan

  • bun test packages/agent-directory packages/workflows packages/webhook-triggers — all green
  • bun run typecheck
  • bun run lint
  • bun run fmt

@TheGreatAxios
TheGreatAxios merged commit b6103e4 into main Sep 17, 2026
3 of 4 checks passed
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