Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

The rate-limit feature now emits an experimental warning during workflow compilation.

Changes

  • Compiler warning emission: Added experimental warning in pkg/workflow/compiler.go when workflowData.RateLimit is configured, following the same pattern as sandbox-runtime, safe-inputs, and plugins features
  • Integration test: Created rate_limit_experimental_warning_test.go to verify warning behavior across different rate-limit configurations

Implementation

// Emit experimental warning for rate-limit feature
if workflowData.RateLimit != nil {
    fmt.Fprintln(os.Stderr, console.FormatWarningMessage("Using experimental feature: rate-limit"))
    c.IncrementWarningCount()
}

Output Example

$ gh aw compile workflow-with-rate-limit.md
⚠ Using experimental feature: rate-limit
✓ workflow-with-rate-limit.md (25.3 KB)

Workflows without rate-limit configuration show no warning.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add warning emission in compiler.go when rate-limit config is present
- Create integration test to verify warning behavior
- Test passes for all scenarios (with/without rate-limit config)
- Warning displayed during compilation: "Using experimental feature: rate-limit"

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Mark rate-limit as experimental in compilation process Mark rate-limit as experimental Feb 12, 2026
Copilot AI requested a review from pelikhan February 12, 2026 02:57
@pelikhan pelikhan marked this pull request as ready for review February 12, 2026 03:08
Copilot AI review requested due to automatic review settings February 12, 2026 03:08
@pelikhan pelikhan merged commit 6684bf9 into main Feb 12, 2026
146 checks passed
@pelikhan pelikhan deleted the copilot/mark-rate-limit-experimental branch February 12, 2026 03:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an “experimental feature” warning for workflows that configure rate-limit, aligning rate-limit with other experimental frontmatter features during compilation.

Changes:

  • Emit a formatted warning to stderr (and increment compiler warning count) when workflowData.RateLimit is set.
  • Add an integration test covering warning emission across multiple rate-limit configurations and the absence of warnings when rate-limit is not configured.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pkg/workflow/compiler.go Emits an experimental warning during validation when rate-limit frontmatter is present.
pkg/workflow/rate_limit_experimental_warning_test.go Integration test verifying the warning appears only when rate-limit is configured.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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