Overview - Adt-cli

adt-cli

7 days30 days

Latest CI Pipeline Executions

Fuzzy

Succeeded
copilot/implement-abapify-pilot-agent
7ce9896c fix(adt-pilot): align declared deps with actual usage - Add `@abapify/adt-client` as a devDependency. Tests import `createAdtClient` from it (tests/workflow.test.ts), and the project reference list in tsconfig.lib.json already points at it. Without an explicit declaration the package was only resolvable via bun workspace hoisting, which fails in stricter resolvers. - Drop `@mastra/mcp` from runtime dependencies. No source file under `src/` imports it; all Mastra surfaces in use are from `@mastra/core/*`, and all MCP transport code is built on `@modelcontextprotocol/sdk`. JSDoc comments referencing `@mastra/mcp` describe an optional consumer choice for sourcing `mcpTools` and remain accurate. Addresses Devin Review findings: - BUG_pr-review-job-5092d9235b51447997e11b8d8766108a_0001 (missing @abapify/adt-client devDep) - BUG_pr-review-job-5092d9235b51447997e11b8d8766108a_0002 (phantom @mastra/mcp runtime dep) Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
by Devin AI
Succeeded
copilot/implement-abapify-pilot-agent
02aae2c5 fix(adt-pilot): use extensionless internal imports per bundler-imports rule Removes .js extensions from all internal relative imports in adt-pilot src and tests to comply with the mandatory bundler-imports rule (since tsconfig.base.json uses moduleResolution: bundler). Matches the convention used in every other package in the monorepo. Addresses Devin Review findings: - BUG_pr-review-job-f33d242b3be645e096ee9c409f2ad9e4_0001 (src/index.ts) - BUG_pr-review-job-f33d242b3be645e096ee9c409f2ad9e4_0002 (src/workflow.ts) - BUG_pr-review-job-f33d242b3be645e096ee9c409f2ad9e4_0003 (src/mcp-client.ts) - BUG_pr-review-job-f33d242b3be645e096ee9c409f2ad9e4_0004 (src/harness.ts) Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
by Devin AI
Succeeded
copilot/implement-abapify-pilot-agent
12af17ef fix(adt-pilot): use https in test/example URLs to clear SonarCloud hotspots SonarCloud (rule typescript:S5332) flagged 'http://sap:8000' string literals in tests/types.test.ts as security hotspots. Switch the example baseUrls in both tests and JSDoc to 'https://sap.example.com' / 'https://my-sap.example.com'. The localhost URL in workflow.test.ts is intentional (in-process mock server) and is not flagged by the rule. Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
by Devin AI
Succeeded
copilot/implement-abapify-pilot-agent
d089159d refactor(adt-pilot): rework workflow + harness modes for 100% accuracy Workflow mode (deterministic, no LLM): - Three-step pipeline: resolveObjects -> runAtcChecks -> buildReport - Discriminated-union input schema with 'package' or 'transport' modes - Package mode enumerates objects via list_package_objects MCP tool - Transport mode validates via cts_get_transport, uses transport URI as ATC target (server-side enumeration matches 'adt check --transport') - Per-object error capture: failed ATC runs become priority='error' findings; the workflow never aborts on a single failure Harness mode (interactive via Mastra Agent): - createAbapifyPilot(config) returns a Mastra Harness with a 'review' mode - Review agent wired with list_package_objects, atc_run, cts_get_transport - Optional custom instructions and pre-loaded MCP tools Type/build hygiene: - Type-erase Mastra workflow + step factories at the import boundary; Mastra's deeply generic types caused tsc OOM and DTS bloat - Public API surfaces narrow CodeReviewWorkflow / CodeReviewRun handles - typecheck target overridden to 'tsc -p tsconfig.lib.json --noEmit' so tsc does not follow project references (avoids adt-mcp typecheck OOM) - Move @abapify/adt-mcp from runtime deps to devDeps (only used in tests) - Promote @modelcontextprotocol/sdk to a runtime dep (used by mcp-client) Tests (28 total, all passing): - workflow.test.ts: rewritten for async createRun() API; covers package mode happy path + empty package, transport mode happy path + URI assertion, single + multi-object error paths, transport-lookup failure - types.test.ts: schema validation; credentials generated via randomBytes - harness.test.ts (new): factory smoke tests for Harness + Agent - mcp-client.test.ts: unchanged (still passing) Security: - All test credentials generated per-process with randomBytes (no more hardcoded 'secret'/'test-password' strings flagged by SonarCloud) Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
by Devin AI
Failed
copilot/implement-abapify-pilot-agent
57afd776 feat(adt-pilot): fix build — disable dts (OOM), explicit return types Agent-Logs-Url: https://github.com/abapify/adt-cli/sessions/8c6e39c2-3ae6-4d7f-8cef-5ff4f1b5904d Co-authored-by: ThePlenkov <6381507+ThePlenkov@users.noreply.github.com>
by copilot-sw...