Skip to content

4.3.0#20023

Open
RobinMalfait wants to merge 7 commits intomainfrom
release/4.3.0
Open

4.3.0#20023
RobinMalfait wants to merge 7 commits intomainfrom
release/4.3.0

Conversation

@RobinMalfait
Copy link
Copy Markdown
Member

No description provided.

@RobinMalfait RobinMalfait requested a review from a team as a code owner May 7, 2026 13:49
Just happened to run into this in the release branch...
```diff
- --- ./dist/assets/index-<hash>.css ---
+ --- ./dist/assets/index-B_aD48aL.css ---
  .do-include-me {
    color: green;
  }
```

For context, when we deal with Vite related integration tests, we build
a CSS file, and dump all the generated files (file path + contents).
Since Vite uses a hash in the filename, we stripped it and replaced it
with `<hash>` because the hash doesn't matter for the tests.

We just happen to run into a situation where a `_` character was used
and our regex didn't match.

Long story short, this is a small internal change that doesn't need any
kind of changelog entry.
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 7, 2026

Confidence Score: 5/5

Safe to merge — this is a straightforward minor-version release that graduates two stable utilities and bumps all package versions consistently.

All packages are consistently bumped to 4.3.0, matching the CHANGELOG entry. The feature-flag removal is complete with no remaining imports anywhere in the codebase. The only artifact is an empty feature-flags.ts file, which compiles and runs fine.

packages/tailwindcss/src/feature-flags.ts is now an empty file and could be deleted outright.

Reviews (2): Last reviewed commit: "bump versions to `4.3.0`" | Re-trigger Greptile

Comment thread packages/tailwindcss/package.json Outdated
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

Review Change Stack

Walkthrough

This release promotes scrollbar utilities and @container size utilities from experimental features to stable by removing their feature flag gates. All package versions are bumped to 4.3.0 across the monorepo, including the core tailwindcss package, platform-specific oxide binaries, and integrations. The feature-flags module exports are cleaned up, and utilities.ts now registers these features unconditionally. Changelog entries document the changes, and test assertion formatting is adjusted for readability.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess its relevance to the changeset. Add a description explaining the 4.3.0 release highlights, such as new features (container-size and scrollbar utilities), bug fixes, and version bumps across packages.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title '4.3.0' is directly related to the changeset as it represents a version bump from 4.2.4 to 4.3.0 across all packages and documentation updates.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
packages/tailwindcss/src/utilities.ts (1)

5940-5941: 💤 Low value

Consider adding size to the suggestions.

Now that @container-size is promoted to stable, the suggestion block should include size as a valid value for autocomplete discoverability.

♻️ Suggested fix
 suggest('@container', () => [
   {
-    values: ['normal'],
+    values: ['normal', 'size'],
     valueThemeKeys: [],
     hasDefaultValue: true,
   },
 ])

Also applies to: 5953-5959

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/tailwindcss/src/utilities.ts` around lines 5940 - 5941, The
suggestions logic currently omits the stable container value "size": update the
branch that checks candidate.value.kind === 'named' and candidate.value.value
=== 'size' to set value = 'size' and ensure any suggestions array or completion
list that handles container query values includes 'size' as an option; apply the
same change to the other occurrence of the same named-value handling (the second
block handling candidate.value.kind === 'named' for container values) so both
places return or push 'size' into the suggestions/completions.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@packages/tailwindcss/src/utilities.ts`:
- Around line 5940-5941: The suggestions logic currently omits the stable
container value "size": update the branch that checks candidate.value.kind ===
'named' and candidate.value.value === 'size' to set value = 'size' and ensure
any suggestions array or completion list that handles container query values
includes 'size' as an option; apply the same change to the other occurrence of
the same named-value handling (the second block handling candidate.value.kind
=== 'named' for container values) so both places return or push 'size' into the
suggestions/completions.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a2e3d18e-350d-4f47-90e9-0c915603c5f8

📥 Commits

Reviewing files that changed from the base of the PR and between c00da89 and a9c8c59.

📒 Files selected for processing (27)
  • CHANGELOG.md
  • crates/node/npm/android-arm-eabi/package.json
  • crates/node/npm/android-arm64/package.json
  • crates/node/npm/darwin-arm64/package.json
  • crates/node/npm/darwin-x64/package.json
  • crates/node/npm/freebsd-x64/package.json
  • crates/node/npm/linux-arm-gnueabihf/package.json
  • crates/node/npm/linux-arm64-gnu/package.json
  • crates/node/npm/linux-arm64-musl/package.json
  • crates/node/npm/linux-x64-gnu/package.json
  • crates/node/npm/linux-x64-musl/package.json
  • crates/node/npm/wasm32-wasi/package.json
  • crates/node/npm/win32-arm64-msvc/package.json
  • crates/node/npm/win32-x64-msvc/package.json
  • crates/node/package.json
  • integrations/vite/resolvers.test.ts
  • packages/@tailwindcss-browser/package.json
  • packages/@tailwindcss-cli/package.json
  • packages/@tailwindcss-node/package.json
  • packages/@tailwindcss-postcss/package.json
  • packages/@tailwindcss-standalone/package.json
  • packages/@tailwindcss-upgrade/package.json
  • packages/@tailwindcss-vite/package.json
  • packages/@tailwindcss-webpack/package.json
  • packages/tailwindcss/package.json
  • packages/tailwindcss/src/feature-flags.ts
  • packages/tailwindcss/src/utilities.ts
💤 Files with no reviewable changes (1)
  • packages/tailwindcss/src/feature-flags.ts

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