Skip to content

Epic: correctness fixes from package-by-package quality sweep #145

Description

@suleimansh

Package-by-package quality review (2026-07-03) surfaced 5 behavior/correctness issues (as opposed to pure refactors, which are deferred). These change behavior or hide failures from callers, so they go first. One issue = one PR = one changeset.

Children

  • connectors: per-connector instructions is silently dropped by mountConnectors
  • connectors: validation branches return success instead of MCP errors
  • connectors: fetch rejections escape as raw TypeError, bypassing the typed error class
  • mcp: resolveOrConstruct swallows resolver errors and builds an unwired instance
  • ai-autopilot: DockerRunner symbols are unreachable from the package entry

Deferred to a later pass (not in this epic): untyped API-payload interfaces, shared cast/helper extractions (lazyModule, makeSafeEmitter, runner/spawn, asTool), and the test-coverage additions (tool-execution, web-handler, runner disk I/O).

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingpriority: highShould be addressed soon

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions