Skip to content

fix: reuse existing pull requests#414

Merged
chubes4 merged 1 commit into
mainfrom
fix-idempotent-create-pr
May 16, 2026
Merged

fix: reuse existing pull requests#414
chubes4 merged 1 commit into
mainfrom
fix-idempotent-create-pr

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 16, 2026

Summary

  • Preflight createPullRequest() for an existing open PR with the same head/base before creating a new one.
  • Return the existing PR with reused: true and skip duplicate PR/artifact side effects when found.
  • Add smoke coverage for the preflight request and reuse path.

Testing

  • php tests/smoke-github-create-abilities.php
  • php -l inc/Abilities/GitHubAbilities.php && php -l tests/smoke-github-create-abilities.php
  • git diff --check

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Drafted the implementation and smoke coverage; Chris remains responsible for review and merge.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 16, 2026

Homeboy Results — data-machine-code

Lint

lint — failed

  • phpstan — 15 finding(s)
  • other — 11 finding(s)
  • formatting — 1 finding(s)
  • Total: 27 finding(s)

ℹ️ Auto-fix: homeboy lint data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since cec6faf --fix (or homeboy refactor data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since cec6faf --from lint --write)
ℹ️ Some issues may require manual fixes
ℹ️ Full options: homeboy docs commands/lint
ℹ️ Save lint baseline: homeboy lint data-machine-code --baseline
Deep dive: homeboy lint data-machine-code --changed-since cec6faf

Test

test — failed

ℹ️ No tests ran — the runner failed before producing results. See raw_output.stderr_tail / raw_output.stdout_tail for the underlying error (bootstrap failure, missing deps, DB connection, etc.).
ℹ️ To run specific tests: homeboy test data-machine-code -- --filter=TestName
ℹ️ Auto-fix lint issues: homeboy refactor data-machine-code --from lint --write
ℹ️ Collect coverage: homeboy test data-machine-code --coverage
ℹ️ Analyze failures: homeboy test data-machine-code --analyze
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine-code --changed-since cec6faf

Audit

audit — passed

  • dead_code — 5 finding(s)
  • intra-method-duplication — 4 finding(s)
  • repeated_literal_shape — 1 finding(s)
  • requested_detectors — 1 finding(s)
  • structural — 1 finding(s)
  • test_coverage — 1 finding(s)
  • Total: 13 finding(s)

Deep dive: homeboy audit data-machine-code --changed-since cec6faf

Tooling versions
  • Homeboy CLI: homeboy 0.182.0+67c9501
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 9eb4c10
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 977c72b into main May 16, 2026
6 of 10 checks passed
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