Skip to content

feat: add adt_rununit OpenCode tool for remote ABAP unit test execution#62

Merged
MagPasulke merged 6 commits into
mainfrom
feat/adt-rununit-tool
May 30, 2026
Merged

feat: add adt_rununit OpenCode tool for remote ABAP unit test execution#62
MagPasulke merged 6 commits into
mainfrom
feat/adt-rununit-tool

Conversation

@MagPasulke
Copy link
Copy Markdown
Owner

Adds a new OpenCode tool adt_rununit that triggers ABAP Unit tests on a remote SAP system via abap-adt-api.

Design

  • Core logic: scripts/adt_rununit_core.ts (reusable)
  • OpenCode tool: .opencode/tools/adt_rununit.ts (wrapper)
  • Parameters: optional objectType (CLAS/DEVC) + objectName. Defaults to SAP_ROOT_PACKAGE env var.
  • Output: Formatted text summary with pass/fail counts, timing, and failure details.

Configuration

Requires in .env:

  • SAP_ADT_URL, SAP_ADT_USER, SAP_ADT_PASSWORD (existing)
  • SAP_ROOT_PACKAGE (new — default package for test runs)

⚠️ Please sync to SAP system via abapGit and run ABAP Unit tests before merging.

…execution

Adds a new OpenCode tool that triggers ABAP Unit tests on a remote SAP
system via abap-adt-api. Follows the same core/wrapper split as adt_gitpull.

- scripts/adt_rununit_core.ts: reusable core logic
- .opencode/tools/adt_rununit.ts: OpenCode tool wrapper
- Supports CLAS and DEVC object types
- Defaults to SAP_ROOT_PACKAGE env var when no target specified
- Returns formatted text summary with pass/fail counts and failure details
@MagPasulke MagPasulke marked this pull request as ready for review May 30, 2026 12:42
The @opencode-ai/plugin tool() function expects Zod schemas for args,
not plain {type, description, required} objects. This caused a
'Bad Request' error when OpenCode tried to load the tool.
@MagPasulke MagPasulke merged commit a5e0b15 into main May 30, 2026
1 check passed
@MagPasulke MagPasulke deleted the feat/adt-rununit-tool branch May 30, 2026 13:01
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