Skip to content

Fix wording inconsistencies in workflow prompts and script help text - #543

Merged
strawgate merged 3 commits into
mainfrom
copilot/fix-wording-inconsistencies
Mar 2, 2026
Merged

strawgate merged 3 commits into
mainfrom
copilot/fix-wording-inconsistencies

Conversation

Copilot AI commented Mar 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This PR fixes wording inconsistencies in workflow prompts and script help text so instructions match actual allowed behavior.

What changed

  • Corrected grammar in both issue-triage prompts: Layout → Lay out.
  • Updated scripts/fetch-workflow-logs.py usage text to include the supported any value for --conclusion.
  • Clarified read-only workflow command guidance to avoid contradictory wording:
    • issue-triage/ro: changed execute commands to execute arbitrary commands, and clarified that only tools/commands listed in (allowed_tools) are available.
    • pr-review/ro: changed execute commands to execute arbitrary commands, and clarified that explicitly listed review helper scripts and read-only git commands are allowed.

Context

Generated by Update PR Body for issue #543

- Fix 'Layout' → 'Lay out' grammar in issue-triage ro and rwx variants
- Add missing 'any' value to --conclusion help text in fetch-workflow-logs.py
- Clarify contradictory command-permission wording in issue-triage/ro
- Clarify contradictory command-permission wording in pr-review/ro

Co-authored-by: strawgate <6384545+strawgate@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix wording inconsistencies in workflow prompts Fix wording inconsistencies in workflow prompts and script help text Mar 2, 2026
@strawgate
strawgate marked this pull request as ready for review March 2, 2026 17:42
@coderabbitai

coderabbitai Bot commented Mar 2, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between beba2a3 and b5d9087.

📒 Files selected for processing (4)
  • claude-workflows/issue-triage/ro/action.yml
  • claude-workflows/issue-triage/rwx/action.yml
  • claude-workflows/pr-review/ro/action.yml
  • scripts/fetch-workflow-logs.py

📝 Walkthrough

Walkthrough

This pull request updates workflow action configuration files and a Python script to improve clarity and fix grammatical errors. Changes include correcting the verb form "Layout" to "Lay out" in two issue-triage workflow variants, clarifying constraint language in read-only workflows by distinguishing between arbitrary and explicitly allowed commands (git operations and helper scripts), tightening documentation about available tools, and adding the "any" value to the --conclusion option in a workflow log-fetching script to accurately reflect supported filtering behavior.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR addresses all four coding requirements from issue #529: fixed 'Layout' → 'Lay out' in both issue-triage variants, added 'any' value to fetch-workflow-logs.py help text, and clarified contradictory command permissions in both issue-triage/ro and pr-review/ro.
Out of Scope Changes check ✅ Passed All changes are directly scoped to fixing wording inconsistencies identified in issue #529; no unrelated modifications detected.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch copilot/fix-wording-inconsistencies

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

@strawgate
strawgate merged commit 4b5316a into main Mar 2, 2026
17 checks passed
@strawgate
strawgate deleted the copilot/fix-wording-inconsistencies branch March 2, 2026 17:58
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.

[text-auditor] Fix wording inconsistencies in workflow prompts and script help text

2 participants