-
Notifications
You must be signed in to change notification settings - Fork 1
Closed
Description
Summary
Compare the egg container's git and gh wrapper scripts (PR #5) against jib-container's versions to ensure all operations are properly handled.
Current State
| Script | jib (lines) | egg (lines) | Difference |
|---|---|---|---|
| scripts/git | 810 | 786 | -24 lines |
| scripts/gh | 519 | 503 | -16 lines |
The differences are small but may indicate missing functionality.
Key Features to Verify
Git Wrapper
- Push with branch ownership policy
- Fetch from remote
- Pull (fetch + merge)
- Clone blocking (with helpful message)
- Init blocking
- Remote modification blocking
- Config handling (global vs local)
- Status, log, diff passthrough
- Commit, checkout, merge operations
- Error message formatting
- Session token authentication
Gh Wrapper
- PR create
- PR comment
- PR edit
- PR close
- PR merge blocking (with helpful message)
- PR view passthrough
- Issue operations
- API passthrough
- Session token authentication
- Error handling
Investigation
Run diff between the scripts:
diff ~/repos/james-in-a-box/jib-container/scripts/git \
~/repos/egg/container/sandbox/scripts/git # (from PR #5)Acceptance Criteria
- Complete audit of script differences
- Document any intentionally different behavior
- Port any missing functionality
- Ensure all error messages are helpful
Authored-by: jib
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels