Remove graduated primer_react_action_list_item_gap feature flag

[Copilot]

Overview

Graduate the primer_react_action_list_item_gap feature flag so NavList item gaps are enabled by default. Remove the flag from the default feature-flag scope and update the related stories, tests, and changelog entry.

Changelog

New

• None.

Changed

• NavList item gaps are now enabled by default.
• Updated NavList stories and tests to reflect the default behavior.

Removed

• The primer_react_action_list_item_gap feature flag.

Rollout strategy

• Patch release
• Minor release
• Major release; if selected, include a written rollout or migration plan
• None; if selected, include a brief description as to why

Testing & Reviewing

• Focused tests: 91 passed
• Full test suite: 2,633 passed, 2 skipped, 10 todo
• Type-check, build, lint, CSS lint, and format checks passed
• CodeQL reported 0 JavaScript alerts

[changeset-bot]

🦋 Changeset detected

Latest commit: 9d98d65

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/react	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

⚠️ Action required

👋 Hi, this pull request contains changes to the source code that github/github-ui depends on. If you are GitHub staff, test these changes with github/github-ui using the integration workflow. Check the integration testing docs for step-by-step instructions. Or, apply the integration-tests: skipped manually label to skip these checks.

[primer-integration]

Integration test results from github/github-ui PR:

  CI Failed
  VRT Running
  Projects Passed

CI check runs linting, type checking, and unit tests. Check the workflow logs for specific failures.

Need help? If you believe this failure is unrelated to your changes, please reach out to the Primer team for assistance.

[Copilot]

🟢 Approval recommended

The flag is fully removed from the runtime path and defaults, and the associated tests/stories/changeset are updated consistently with no remaining in-code references.

Pull request overview

This PR graduates the primer_react_action_list_item_gap feature flag so NavList item spacing is enabled by default, and removes the flag from the default feature flag registry. It also updates tests, stories, and adds a changeset for a patch release.

Changes:

• Enable NavList item gaps by default by removing the feature-flag gate in ActionList when container === 'NavList'.
• Update NavList and ActionList tests and NavList feature stories to reflect the new default behavior (and remove FeatureFlags wrappers for this behavior).
• Remove primer_react_action_list_item_gap from DefaultFeatureFlags and add a patch changeset.

File summaries

File	Description
packages/react/src/NavList/NavList.test.tsx	Updates assertions to expect item gaps by default and removes feature-flag-specific cases.
packages/react/src/NavList/NavList.features.stories.tsx	Removes feature-flag wrapper and renames the story to reflect default behavior.
packages/react/src/FeatureFlags/DefaultFeatureFlags.ts	Removes the graduated flag from the default flag scope.
packages/react/src/ActionList/List.tsx	Makes data-item-gap unconditional for NavList container (no feature flag).
packages/react/src/ActionList/ActionList.test.tsx	Removes now-obsolete test for “flag enabled” behavior while keeping standalone default behavior coverage.
.changeset/nav-list-item-gaps.md	Adds patch changeset documenting the default behavior change.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 0
• Review effort level: Lite

---

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