Skip to content

Tell agents to format changed files with Prettier - #5187

Open
AbuBakkarBinAkmal wants to merge 1 commit into
temporalio:mainfrom
AbuBakkarBinAkmal:ab/document-prettier-workflow
Open

Tell agents to format changed files with Prettier#5187
AbuBakkarBinAkmal wants to merge 1 commit into
temporalio:mainfrom
AbuBakkarBinAkmal:ab/document-prettier-workflow

Conversation

@AbuBakkarBinAkmal

Copy link
Copy Markdown

Summary

  • require agents to run Prettier 3 on every changed file it supports
  • use the existing .prettierrc without command-line overrides
  • limit formatting to explicit paths and require diff inspection to avoid unrelated churn

Historical context

The repository previously removed whole-repository formatter automation after it changed unrelated MDX. This keeps #5153 scoped to agent instructions and does not add CI auto-commits or another formatter dependency.

Validation

  • npx --yes prettier@3 --write .prettierrc
  • npx --yes prettier@3 --check .prettierrc
  • compared the Prettier-normalized block with upstream/main; only the new guidance differs
  • git diff --check upstream/main...HEAD

Closes #5153

Copilot AI balanced review requested due to automatic review settings August 26, 2026 20:00
@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

@AbuBakkarBinAkmal is attempting to deploy a commit to the Temporal Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

CLAassistant commented Aug 26, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds agent instructions for scoped Prettier 3 formatting using the repository configuration.

Changes:

  • Requires formatting and checking supported changed files.
  • Prevents broad formatting and unrelated churn.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@AbuBakkarBinAkmal
AbuBakkarBinAkmal marked this pull request as ready for review August 26, 2026 20:04
@AbuBakkarBinAkmal
AbuBakkarBinAkmal requested a review from a team as a code owner August 26, 2026 20:04
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.

Tell agents to use our prettier config

3 participants