Skip to content

Internal: clean up AGENTS.md - #10034

Merged
samohovets merged 1 commit into
mainfrom
agents-md-improve
Jul 31, 2026
Merged

Internal: clean up AGENTS.md#10034
samohovets merged 1 commit into
mainfrom
agents-md-improve

Conversation

@samohovets

Copy link
Copy Markdown
Member

Summary

  • add coding-style guidance that keeps logic together and avoids preemptive single-use helper extraction
  • remove workflow instructions already owned by dedicated skills or standard repository conventions
  • remove obsolete Cloud Agent-specific setup and caveats while retaining platform-independent service and rendering commands

Rationale

  • Keeping logic in one function unless it is composable or reusable avoids unnecessary indirection. Single-use helpers should stay inline unless they hide a genuinely complex boundary or have a clear independent name that makes the caller easier to understand.
  • The pre-commit build and style-check instructions duplicate the pr skill, so the skill should remain the authoritative workflow. The generic bun.lock reminder was removed with that redundant checklist to keep AGENTS.md focused.
  • Pull request title formatting duplicates the pr-name skill, which already inspects and classifies each change before choosing a title.
  • Agents are expected to discover contributing documentation through the repository's standard conventions. The generic Contributing section was not being consulted during feature work or PR submission, so keeping it in AGENTS.md did not improve behavior.
  • Development no longer happens in Cloud Agents; it happens on developer Macs. Cloud-specific runtime setup and known VM caveats are therefore obsolete.
  • Key local services and video-rendering commands remain useful on developer machines, so they are retained as top-level sections.

Verification

  • bunx oxfmt AGENTS.md --write
  • bun run build
  • bun run stylecheck

@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview Jul 31, 2026 3:52pm
remotion Ready Ready Preview Jul 31, 2026 3:52pm

Request Review

@pullfrog pullfrog Bot 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.

✅ No new issues found.

Reviewed changes

  • Coding style section: Added guidance to keep logic in one function and avoid preemptive single-use helper extraction.
  • Removed redundant sections: Before committing, Contributing, and PR title format are already covered by the pr and pr-name skills.
  • Removed Cloud Agent-specific content: Obsolete runtime instructions, known VM caveats, and platform-specific setup.
  • Promoted Key services and Rendering test videos: Elevated from Cloud Agent subsections to top-level headings.

The removed Before committing steps (bun run build, bun run stylecheck) are present in the pr skill at .agents/skills/pr/SKILL.md:22-24. The PR title format is defined in .agents/skills/pr-name/SKILL.md:13-14 with additional classification rules for special prefixes. The restructuring is internally consistent.

Pullfrog  | View workflow run | Using DeepSeek Pro (free via Pullfrog for OSS) (Claude Opus not used — the program covers this model; add its provider key to run your pick) | 𝕏

@samohovets samohovets changed the title Internal: Streamline coding agent guidance Internal: clean up AGENTS.md Jul 31, 2026
@samohovets
samohovets merged commit 217e07f into main Jul 31, 2026
17 checks passed
@samohovets
samohovets deleted the agents-md-improve branch July 31, 2026 16:25
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