Skip to content

docs(agents): add GitHub Actions workflow guidance - #17896

Open
ChronosSF wants to merge 1 commit into
masterfrom
sstoychev/agents-md-workflows
Open

ChronosSF wants to merge 1 commit into
masterfrom
sstoychev/agents-md-workflows

Conversation

@ChronosSF

@ChronosSF ChronosSF commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Description

Adds a short GitHub Actions workflows section to the Working Rules in AGENTS.md:

  • Pin every action and reusable workflow to a full commit SHA with a # vX.Y.Z comment.
  • Use each action's current inputs instead of copying older workflows. For example, actions/create-github-app-token takes client-id: ${{ secrets.IGNITEUI_GITHUB_APP_CLIENT_ID }}; app-id is deprecated since v3.1.0.
  • Deploys to Infragistics infrastructure go through a repository_dispatch to IgniteUI/igniteui-actions.
  • Validate changed workflows with actionlint.

Motivation / Context

The review of the SassDoc deploy (#17891 and its igniteui-actions counterpart) found two problems in the agent-written workflow:

  • It used the deprecated app-id input, copied from existing workflows.
  • It referenced a reusable workflow by @master instead of a SHA, which code scanning flags.

AGENTS.md had no guidance on workflows, and its "reuse existing repository patterns" rule pointed agents straight at the deprecated input: npm-publish.yml and trigger-licensed-release.yml still use app-id.

The same guidance goes to IgniteUI/igniteui-actions in its own AGENTS.md, together with the pin and client-id fixes there.

Backports: 22.2.x #17897, 22.1.x #17898, 22.0.x #17899.

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • CI/CD
  • Tests
  • Changelog
  • Skills/Agents

Component(s) / Area(s) Affected:

AGENTS.md (CI workflows)

How Has This Been Tested?

  • Unit tests
  • Manual testing
  • Automated e2e tests

Documentation only.

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

🤖 Generated with Claude Code

Pin actions to commit SHAs, use the current create-github-app-token
inputs (client-id) instead of copying older workflows, dispatch deploys
to IgniteUI/igniteui-actions and validate workflows with actionlint.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

This branch has not been deployed

No deployments
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