Skip to content

Comments

Add AGENTS.md and GitHub Copilot setup workflow#5556

Merged
Boshen merged 3 commits intomainfrom
copilot/fix-b2c1fb3c-e3e7-4536-9b00-a7d1b63f01dc
Aug 2, 2025
Merged

Add AGENTS.md and GitHub Copilot setup workflow#5556
Boshen merged 3 commits intomainfrom
copilot/fix-b2c1fb3c-e3e7-4536-9b00-a7d1b63f01dc

Conversation

Copy link
Contributor

Copilot AI commented Aug 2, 2025

This PR adds comprehensive AI agent support to the Rolldown project by introducing documentation and automation for GitHub Copilot and other AI assistants working on the codebase.

Files Added

AGENTS.md

A comprehensive guide for AI agents working on Rolldown that includes:

  • Project Overview: Clear explanation of Rolldown's architecture, tech stack, and purpose as a Rust-based bundler for Vite
  • Development Guidelines: Best practices for AI agents including minimal change philosophy, preservation of working code, and iterative testing approaches
  • Build & Test Workflows: Reference for essential commands using Just, Cargo, and pnpm
  • Code Conventions: Formatting standards for Rust (rustfmt) and TypeScript/JavaScript (dprint)
  • Common Patterns: Examples of typical code structures used in the codebase
  • Resource Links: Quick access to documentation, contributing guides, and community resources

.github/workflows/copilot-setup-steps.yml

A GitHub Actions workflow that validates and sets up AI agent environments with three modes:

  • Basic Setup: Validates project structure and essential files for AI compatibility
  • Advanced Setup: Tests build system functionality and code formatting tools
  • Agent Configuration: Generates project summaries and configuration details for AI agents

The workflow automatically triggers on:

  • Manual dispatch with configurable setup types
  • Pushes to copilot/** branches when AI agent files are modified

Benefits

This addition provides several key advantages:

  1. Standardized AI Interaction: Clear guidelines help AI agents understand project conventions and make appropriate contributions
  2. Quality Assurance: Automated validation ensures AI agents have access to properly configured development environments
  3. Reduced Onboarding Time: New AI contributors can quickly understand the project structure and development workflow
  4. Consistency: Enforces the same high standards for AI-generated code as human contributions

The implementation follows existing project patterns, uses the same formatting standards (80-character line limits), and integrates seamlessly with the current CI/CD infrastructure.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dprint.dev
    • Triggering command: curl -fsSL REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@netlify
Copy link

netlify bot commented Aug 2, 2025

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 9d7e8ca
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/688e0f1aee5fed0008bb94ac

Copilot AI changed the title [WIP] Add AGENTS.md and .github/workflows/copilot-setup-steps.yml Add AGENTS.md and GitHub Copilot setup workflow Aug 2, 2025
Copilot AI requested a review from Boshen August 2, 2025 03:31
@Boshen Boshen marked this pull request as ready for review August 2, 2025 13:17
@Boshen Boshen merged commit 0736023 into main Aug 2, 2025
22 checks passed
@Boshen Boshen deleted the copilot/fix-b2c1fb3c-e3e7-4536-9b00-a7d1b63f01dc branch August 2, 2025 13:17
name: Copilot Setup Steps

# This workflow defines the setup steps that GitHub Copilot agents will use
# to prepare the development environment for the oxc project.
Copy link
Contributor

Choose a reason for hiding this comment

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

The workflow references "oxc project" in the comment, but this PR is for the Rolldown project. This appears to be a copy-paste error that should be corrected to maintain consistency with the rest of the documentation.

Suggested change
# to prepare the development environment for the oxc project.
# to prepare the development environment for the Rolldown project.

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

This was referenced Aug 4, 2025
shulaoda added a commit that referenced this pull request Aug 4, 2025
## [1.0.0-beta.31] - 2025-08-04

### 🚀 Features

- support function for asset_inline_limit (#5453) by @AliceLanniste
- hmr: generate initializer for `export * from ...` (#5577) by @hyf0
- hmr: remove meaningless stmt instead of replacing with empty stmt (#5570) by @hyf0
- option `watch.clearScreen` (#5518) by @situ2001
- rolldown: oxc v0.80.0 (#5538) by @Boshen
- plugin/generateBundle: add support for modifying chunk filename (#5564) by @shulaoda
- assetFileNames: support `name` and `originalFileName` (#5555) by @shulaoda
- add optional `onInvalidate` schema to validator (#5528) by @situ2001
- don't throw resolve errors for dynamic import inside try block (#5537) by @shulaoda
- rolldown: oxc v0.79.0 (#5512) by @Boshen
- rolldown_plugin_asset: align `render_chunk` logic (#5511) by @shulaoda
- rolldown_plugin_utls: finish `render_asset_url_in_js` (#5508) by @shulaoda
- rolldown_plugin_utils: align replacement logic for `__VITE_ASSET__` (#5507) by @shulaoda
- rolldown_plugin_utils: add partial `render_asset_url_in_js` support (#5504) by @shulaoda
- expose `sourcemapBaseUrl` from NormalizedOutputOptions (#5513) by @sapphi-red

### 🐛 Bug Fixes

- modify output filename correctly in `generateBundle` and `writeBundle` (#5580) by @shulaoda
- hmr: ensure correct runtime behavior for `export * from ...` (#5578) by @hyf0
- hmr: repeated imports should point to the same binding (#5571) by @hyf0
- hmr: handle `export * from ...` (#5566) by @hyf0
- rolldown_plugin_web_worker_post: handle `import.meta` correctly (#5565) by @shulaoda
- antd bundle output not correct (#5549) by @IWANABETHATGUY
- hmr: module namespace object should be included via treeshaking mechanism normally (#5543) by @hyf0
- plugin/vite-resolve: use `import` / `require` conditions for external modules resolution (#5540) by @sapphi-red
- rolldown_plugin_asset: InvalidAsset#is should match InvalidAsset::True (#5529) by @AliceLanniste
- __name lacks require('./chunk-containing-runtime') when format: 'cjs' + keepNames: true (#5526) by @IWANABETHATGUY
- plugin/vite-resolve: resolve externalized packages with `resolve.externalConditions` (#5491) by @sapphi-red
- optimization.inlineConst: true does not keep export names when output.minifyInternalExports: true is enabled (#5520) by @IWANABETHATGUY
- scanner: identify `module.exports.__esModule = true` (#5503) by @overlookmotel
- error instead of panic if `sourcemapBaseUrl` is an invalid URL (#5514) by @sapphi-red

### 🚜 Refactor

- hmr: use `__rolldown_exports__` as the name for esm module namespace object (#5579) by @hyf0
- tests: rename `_configName` to `_testName` in configuration files and update variant formatting in snapshots (#5569) by @hyf0
- rolldown: remove unnecessary comments (#5552) by @situ2001
- rolldown: Remove `rolldown_rstr` crate and use `CompactStr` directly (#5542) by @Copilot
- hide namespace statement index knowledge inside StmtInfos (#5545) by @sapphi-red
- use pascal case when naming bitflags field for `ImportRecordMeta` (#5539) by @IWANABETHATGUY
- use concat_string! to simplify string concatenation (#5527) by @IWANABETHATGUY
- use combinator function handle `sourcemap_base_url` parsing (#5517) by @IWANABETHATGUY
- rolldown_plugin_utils: improve `to_output_file_path_in_js` (#5505) by @shulaoda

### 📚 Documentation

- wording nit (#5521) by @nullxone

### 🧪 Testing

- rust: rename `configName` to `_configName` and add comments (#5568) by @hyf0
- rust: re-organize `rolldown_testing_config` (#5567) by @hyf0

### ⚙️ Miscellaneous Tasks

- deps: lock file maintenance npm packages (#5582) by @renovate[bot]
- deps: lock file maintenance (#5581) by @renovate[bot]
- deps: lock file maintenance rust crates (#5576) by @renovate[bot]
- deps: lock file maintenance npm packages (#5574) by @renovate[bot]
- deps: update github-actions (#5573) by @renovate[bot]
- deps: update dependency rolldown-plugin-dts to v0.15.1 (#5572) by @renovate[bot]
- deps: update dependency tsdown to v0.13.2 (#5562) by @renovate[bot]
- deps: update dependency rolldown-plugin-dts to ^0.15.0 (#5563) by @renovate[bot]
- ai: Add AGENTS.md and GitHub Copilot setup workflow (#5556) by @Copilot
- deps: update dependency tsdown to v0.13.1 (#5551) by @renovate[bot]
- deps: update dependency rolldown-plugin-dts to v0.14.3 (#5544) by @renovate[bot]
- ensure all diagnostics in self.errors have Severity::Error (#5494) by @sapphi-red
- vscode: remove `code-spell-checker` extension (#5530) by @shulaoda
- deps: update dependency rolldown-plugin-dts to v0.14.2 (#5522) by @renovate[bot]
- add link to the REPL in the reproduction field description (#5519) by @sapphi-red
- enable some rollup tests (#5515) by @sapphi-red

### ❤️ New Contributors

* @Copilot made their first contribution in [#5556](#5556)
* @nullxone made their first contribution in [#5521](#5521)

Co-authored-by: shulaoda <[email protected]>
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.

2 participants