Skip to content

Remove informational workflow comments that cascade into extra Actions runs #151

Description

@Mpfk

Summary

Follow-up to #149. Issue/PR comments posted by automation workflows are themselves issue events, which trigger issue-state-guard.yml and potentially issue-native-automation.yml — adding cascading runs for every status transition. The comments are also redundant: agents and developers can observe the same information through labels, CI check status, and branch existence directly.

Description

Remove informational comment-posting from two workflows:

ci-issue-gate.yml

  • Remove: CI failure comment ("Gate check failed: CI checks are not green…")
  • Remove: CI pass comment ("Review kickoff: CI checks are green…")
  • Keep: setStatus() label transitions (these are the real work)

issue-native-automation.yml

  • Remove: "Issue is ready. Branch ready: issue/N. Assign to Copilot…" (branch existence is observable)
  • Remove: "Implementation kickoff:" block (content lives in .github/copilot-instructions.md)
  • Remove: "Gate 1 approved by @Actor…" confirmation (label transition to status/ready is observable)
  • Keep: error-guard comments ("Copilot assignment detected, but issue is not status/ready", "Plan approval command ignored:") — these signal misconfiguration and are not routine

Acceptance Criteria

  • ci-issue-gate.yml no longer posts any createComment calls; label transitions (setStatus) are unchanged
  • issue-native-automation.yml drops the three informational comments but retains the two error-guard comments
  • tests/workflow-config.sh passes with no regressions (all existing assertions green)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions