Skip to content

Audit git/gh wrapper scripts for completeness #43

@james-in-a-box

Description

@james-in-a-box

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions