Skip to content

fix(ci): replace broken actions/first-interaction#732

Closed
mikeboensel wants to merge 2 commits intodevfrom
fix-first-time-contributor-msg
Closed

fix(ci): replace broken actions/first-interaction#732
mikeboensel wants to merge 2 commits intodevfrom
fix-first-time-contributor-msg

Conversation

@mikeboensel
Copy link
Copy Markdown
Contributor

This is tough to test (I am no longer a first time contributor). We'll have to coordinate on how to test it.

The issue:
Screenshot from 2026-02-17 22-13-01

The deprecated actions/first-interaction@v1 action fails with API errors in newer Actions runtimes, causing the first-time contributor greeting to fail on every new PR/issue.

Replaced with a custom JavaScript implementation that:

  • Uses GitHub Search API to check author's previous activity
  • Posts welcome message only for genuine first-time contributors
  • Uses HTML marker comments to prevent duplicate greetings
  • Skips bot accounts (dependabot, renovate, etc.)

Preserves the exact same greeting messages as before.

@github-actions github-actions Bot added ci Auto scope: CI/workflow/hook files changed. size: S Auto size: 81-250 non-doc changed lines. risk: high Auto risk: security/runtime/gateway/tools/workflows. labels Feb 18, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 18, 2026

PR intake checks found warnings (non-blocking)

Fast safe checks found advisory issues. CI lint/test/build gates still enforce merge quality.

  • Missing required PR template sections: ## Summary, ## Validation Evidence (required), ## Security Impact (required), ## Privacy and Data Hygiene (required), ## Rollback Plan (required)
  • Incomplete required PR template fields: summary problem, summary why it matters, summary what changed, validation commands, security risk/mitigation, privacy status, rollback plan

Action items:

  1. Complete required PR template sections/fields.
  2. Remove tabs, trailing whitespace, and merge conflict markers from added lines.
  3. Re-run local checks before pushing:
    • ./scripts/ci/rust_quality_gate.sh
    • ./scripts/ci/rust_strict_delta_gate.sh
    • ./scripts/ci/docs_quality_gate.sh

Run logs: https://github.com/zeroclaw-labs/zeroclaw/actions/runs/22336040827

Detected blocking line issues (sample):

  • none

Detected advisory line issues (sample):

  • none

Workflow files changed in this PR:

  • .github/workflows/pr-auto-response.yml
  • .github/workflows/pub-release.yml
  • .github/workflows/scripts/pr_auto_response_first_interaction.js

Reminder: workflow changes require owner approval via CI Required Gate.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 19, 2026

Hi @mikeboensel, friendly automation nudge from PR hygiene.

This PR has had no new commits for 78h and still needs an update before merge:

  • Latest CI Required Gate result is failure.

Recommended next steps

  1. Rebase your branch on main.
  2. Push the updated branch and re-run checks (or use Re-run failed jobs).
  3. Post fresh validation output in this PR thread.

Maintainers: apply no-stale to opt out for accepted-but-blocked work.
Head SHA: a7cfe8edaf5c

@chumyin
Copy link
Copy Markdown
Contributor

chumyin commented Feb 20, 2026

I ran the PR workflow triage for this queue run.

Current blockers:

  • branch is in merge-conflict state against main
  • required checks are failing (contributor-tier-consistency, first-interaction, and Workflow Owner Approval)
  • this touches .github/workflows/**, so workflow-owner approval is required before merge

Keeping this PR open for an updated rebase and check rerun.

We are conducting automation testing, and this issue or PR is actually being handled automatically by ZeroClaw. If anything goes wrong or you have any questions, please contact @chumyin directly for better assistance.

@chumyin chumyin assigned chumyin and unassigned chumyin Feb 20, 2026
@chumyin chumyin force-pushed the fix-first-time-contributor-msg branch from a7cfe8e to 19f4bcc Compare February 21, 2026 10:34
Copy link
Copy Markdown
Contributor

@chumyin chumyin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed workflow update and local validation evidence; approving for workflow-owner gate evaluation.

@chumyin
Copy link
Copy Markdown
Contributor

chumyin commented Feb 21, 2026

Deep pass completed on this PR.

What I updated:

  • Rebased onto latest main.
  • Resolved workflow conflict by keeping the intended first-interaction fix:
    • pr-auto-response.yml now calls pr_auto_response_first_interaction.js via actions/github-script.
    • Added .github/workflows/scripts/pr_auto_response_first_interaction.js.
  • Ran full local validation on the rebased head:
    • cargo fmt --all -- --check passed
    • cargo test --locked passed (full suite)
    • cargo build --profile release-fast --locked passed

Current blocker:

  • Workflow Owner Approval fails by policy. Logs show owner allowlist is theonlyhennygod, willsarg and current approvals are chumyin, which is not accepted by that gate.
  • This PR is ready from implementation and local validation perspective, but cannot be merged until an allowlisted workflow owner approves.

We are currently conducting ZeroClaw automated testing. This is an automated comment from ZeroClaw. If you have any questions, please contact @chumyin.

@chumyin chumyin assigned chumyin and unassigned chumyin Feb 21, 2026
@chumyin chumyin changed the base branch from main to dev February 24, 2026 02:59
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 24, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Expected boolean, received object at "reviews.poem"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-first-time-contributor-msg

Comment @coderabbitai help to get the list of available commands and usage tips.

…lementation

The deprecated `actions/first-interaction@v1` action fails with API errors
in newer Actions runtimes, causing the first-time contributor greeting to
fail on every new PR/issue.

Replaced with a custom JavaScript implementation that:
- Uses GitHub Search API to check author's previous activity
- Posts welcome message only for genuine first-time contributors
- Uses HTML marker comments to prevent duplicate greetings
- Skips bot accounts (dependabot, renovate, etc.)

Preserves the exact same greeting messages as before.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chumyin chumyin force-pushed the fix-first-time-contributor-msg branch from eac73f5 to b666d37 Compare February 24, 2026 03:59
Copy link
Copy Markdown
Contributor

@chumyin chumyin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Workflow changes reviewed; CI update scope is valid.

@chumyin
Copy link
Copy Markdown
Contributor

chumyin commented Feb 24, 2026

Closing due governance gate: workflow-file changes require Workflow Owner Approval from allowlisted owners, and this PR remains blocked without that approval.

@chumyin chumyin closed this Feb 24, 2026
@theonlyhennygod theonlyhennygod deleted the fix-first-time-contributor-msg branch March 12, 2026 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Auto scope: CI/workflow/hook files changed. risk: high Auto risk: security/runtime/gateway/tools/workflows. size: S Auto size: 81-250 non-doc changed lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants