Skip to content

feat: enhance adt_gitpull to support pulling from different branches#65

Merged
MagPasulke merged 22 commits into
mainfrom
feat/gitpull-branch-support
May 30, 2026
Merged

feat: enhance adt_gitpull to support pulling from different branches#65
MagPasulke merged 22 commits into
mainfrom
feat/gitpull-branch-support

Conversation

@MagPasulke
Copy link
Copy Markdown
Owner

Closes #63

Changes

  • Add optional branch parameter to adtGitPull core function
  • Auto-detect branch from current git checkout when not explicitly provided
  • Always pass explicit branch (full ref format) to gitPullRepo API
  • Update OpenCode tool schema with optional branch parameter
  • Update skill documentation

Design Decisions

  • Branch is always passed explicitly to the API (no relying on SAP-side default)
  • Auto-detect via git rev-parse --abbrev-ref HEAD, formatted as refs/heads/<name>
  • CI script unchanged (auto-detect resolves to main since workflow checks out main)
  • Success result returns the branch actually pulled

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

… checkout

- Add optional branch field to AdtGitPullOptions interface
- Auto-detect branch via git rev-parse when not explicitly provided
- Always pass explicit refs/heads/<branch> to gitPullRepo API
- Add branch parameter to OpenCode tool schema
- Update skill documentation with new parameter table

Closes #63
@MagPasulke MagPasulke marked this pull request as ready for review May 30, 2026 19:22
MagPasulke added 19 commits May 30, 2026 21:24
Adds 'npm run typecheck' script that runs tsc --noEmit against
scripts/*.ts and .opencode/tools/*.ts. Included in 'npm test' so
CI catches TypeScript errors in tooling code.
All tests use either the ABAP transpiler runner or Node's built-in
test runner (node --test). Jest was never used.
- Remove redundant 'npm run lint && npm test' (lint runs twice)
- Add typecheck script to the test table
- Simplify to just 'npm test' everywhere
The pullRepo API alone does not change the active branch on SAP.
Use switchRepoBranch first when the target branch differs from
the repo's current configured branch.
@MagPasulke MagPasulke merged commit 5b1fc4c into main May 30, 2026
1 check failed
@MagPasulke MagPasulke deleted the feat/gitpull-branch-support branch May 30, 2026 20:06
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.

feat: enhance adt_gitpull to support pulling from different branches

1 participant