Skip to content

fix: skip actor lookup for fresh issue comments#487

Merged
chubes4 merged 1 commit into
mainfrom
fix-issue-comment-guard-app-token
May 31, 2026
Merged

fix: skip actor lookup for fresh issue comments#487
chubes4 merged 1 commit into
mainfrom
fix-issue-comment-guard-app-token

Conversation

@chubes4

@chubes4 chubes4 commented May 31, 2026

Copy link
Copy Markdown
Member

Summary

  • Avoids calling GET /user before commenting on an issue that has no existing comments.
  • Keeps the automation repeat-comment guard intact for issues that already have comments, where actor comparison is actually needed.

Verification

  • php tests/smoke-github-create-abilities.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Diagnosed GitHub App issue-comment failures from E2E artifacts, implemented the guard ordering fix, and added smoke coverage; Chris remains responsible for review and merge.

@homeboy-ci

homeboy-ci Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — data-machine-code

Lint

lint — passed

ℹ️ Full options: homeboy docs commands/lint
ℹ️ Save lint baseline: homeboy lint data-machine-code --baseline
Deep dive: homeboy lint data-machine-code --changed-since 84a8c04

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-lint-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26723657883

Test

test — passed

ℹ️ Auto-fix lint issues: homeboy refactor data-machine-code --from lint --write
ℹ️ Collect coverage: homeboy test data-machine-code --coverage
ℹ️ Save test baseline: homeboy test data-machine-code --baseline
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine-code --changed-since 84a8c04

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-test-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26723657883

Audit

audit — passed

  • audit — 12 finding(s)
  • Total: 12 finding(s)

Deep dive: homeboy audit data-machine-code --changed-since 84a8c04

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-audit-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26723657883
Tooling versions
  • Homeboy CLI: homeboy 0.213.8+8354a25
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 3dc5eafe
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 2658614 into main May 31, 2026
5 checks passed
@chubes4 chubes4 deleted the fix-issue-comment-guard-app-token branch May 31, 2026 20:34
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