[JacobStephens2]

Summary

The walking skeleton: thirdshift <Issue URL> takes an open issue to a PR on the happy path.

run(Issue URL)
  parse Issue URL; Origin match on raw remote.origin.url
  Base branch = checked-out branch
  Worktree::create_fresh          # ../<repo>-issue-<n>, new issue-<n> from origin/<base>
  Plugin::write                   # embedded skills/ + .claude-plugin/plugin.json (name: thirdshift) in a temp dir
  session::run                    # claude -p --permission-mode auto --plugin-dir <tmp>
                                  #   --output-format stream-json --verbose <fresh prompt>
                                  #   stream → ~/.thirdshift/logs/<owner>-<repo>-issue-<n>-<ts>-implement.jsonl
  git push origin issue-<n>       # even if the agent already pushed
  gh pr view issue-<n>            # exists, OPEN, targets the Base branch
  print PR URL                    # the only stdout
  drop: worktree removed, local issue-<n> deleted, temp dir removed

src/          main, run (orchestration), issue (Issue URL + Origin match), worktree,
              plugin, session (claude + log path), prompt, github (gh), git
tests/
├── support/  harness: real git (bare origin via insteadOf), HOME/TMPDIR per test
├── fakes/    gh (JSON state file), claude (scripted per test, records argv/prompt/cwd/plugin dir)
└── fresh_run.rs
.github/workflows/ci.yml   fmt --check, clippy -D warnings, cargo test

Tested seam: the thirdshift CLI only. Tests run the compiled binary and assert on stdout, exit code, the origin repo, the fake GitHub state, and what the fake claude was asked to do.

Evidence

• Before: prints_only_the_pr_url_and_exits_zero fails, left: "Hello, world!\n", right: "https://github.com/acme/widgets/pull/1\n"
  After: cargo test: 8 passed

  prints_only_the_pr_url_and_exits_zero
  runs_claude_headless_in_auto_mode_in_a_sibling_worktree_on_the_issue_branch
  gives_the_agent_the_fresh_prompt
  loads_every_factory_skill_as_the_thirdshift_plugin
  logs_the_session_stream_under_the_home_directory
  pushes_the_issue_branch_when_the_agent_did_not
  succeeds_when_the_agent_pushed_the_branch_itself
  leaves_no_worktree_local_issue_branch_or_temp_directory_behind
  

• The generated plugin layout passes claude plugin validate.

Unaddressed findings

Standards

• Primitive Obsession (PR state as String, Issue branch built with format!): left for Continuation of an existing Issue branch #6/Numbered Issue branches after a merged or closed PR #7, which reshape branch selection and PR handling.
• Data Clumps (issue, base, branch travelling together): a Run context type is better shaped once Continuation and Repairs add their fields.
• Duplicated Command + status-check shape in git, github, session: three small call sites with different output handling; not worth a shared helper yet.
• Feature Envy (log_path, prompt::fresh read IssueUrl fields), generic run names, Worktree::path() alongside git(): minor, kept for readability.
• Dependencies (current_dir, clock, HOME) created rather than injected: the only seam is the CLI, so they are covered end to end.

Spec

• Origin match already handles SSH, .git and casing, and /issues/<n> parsing and a usage message exist, beyond "basic Origin match only": kept as the spec's definition; Pre-flight checks #4 adds the tests.
• Wrong-base, closed and missing-PR checks are implemented but untested: covered by The Failed run path #5's failure-path tests, which also add isDraft.
• Fake gh state has no issues or check runs yet: Pre-flight checks #4 and CI watching, CI-fix Repair and the Repair cap #9 add them.

Merge Danger

Door: two-way

First code in the repo; nothing depends on it yet. Revert is clean.

Blast Radius: none

Not safe for unattended use until #4 and #5: with no Failed run path, a failed session's unpushed work is deleted in cleanup.

Closes #3

🤖 Generated with Claude Code