Skip to content

AgentA@Area54: ci: "Generated with" must be followed by text on the same line - #56

Merged
muxagent[bot] merged 1 commit into
mainfrom
agenta/generated-with-same-line
Oct 7, 2026
Merged

muxagent[bot] merged 1 commit into
mainfrom
agenta/generated-with-same-line

Conversation

@muxagent

@muxagent muxagent Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #55. It addresses Codex's P2 on #55 (.github/workflows/no-coauthor-trailers.yml:25), which applies to every copy of the check.

The "Generated with" alternative used \s+ after the phrase, and \s also matches a newline. So a description or commit with a heading such as ## Generated with followed by an ordinary paragraph matched, and the required check failed with no attribution on that line. The pattern now uses [ \t]+, so the attribution text must be on the same line. The self-test gains that case as a negative control.

Verified locally: the self-test passes (6 cases). The new pattern still matches the real "Generated with" lines from agentmux #4448–#4450, and no longer matches the heading case.

Agent: AgentA@Area54

@muxreview muxreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude

muxreview Diagnostics
Field Value
muxreview Version 5.34.9+gda231da
Trigger PR opened
Tier STANDARD
Project Context No CLAUDE.md
Model claude-sonnet-5-5
Effort high
Ref Repos Enabled (dev-tools, shared-infrastructure)
Merge Safety No recent-code deletions
Review Time 13s
Timestamp 2026-10-07T19:32:14Z
Repository agentmuxai/muxcode
PR #56

LGTM

@a5af

a5af commented Oct 7, 2026

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T19:33:57.677257Z c53d01b Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: c53d01bdfd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@muxagent
muxagent Bot merged commit c4e9238 into main Oct 7, 2026
3 checks passed
@muxagent
muxagent Bot deleted the agenta/generated-with-same-line branch October 7, 2026 19:35
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