Skip to content

(janitor/dependencies) Remove unused tailwindcss-animate devDependency - #6676

Merged
pandemicsyn merged 1 commit into
mainfrom
janitor/dependencies-storybook-unused-devdeps
Sep 25, 2026
Merged

pandemicsyn merged 1 commit into
mainfrom
janitor/dependencies-storybook-unused-devdeps

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Dependencies removed

  • tailwindcss-animate@1.0.7 — removed from apps/storybook/package.json (devDependencies)

Evidence it was unused

  • No import, require, dynamic import, or type reference anywhere in the repository (grep -rn "tailwindcss-animate" returns only the package.json entry and a stale fixture at apps/web/src/tests/req_sample/anthropic-claude37.log.req.json, which is a historical sample of an old backend package.json, not a consumer).
  • The project already migrated animation utilities to the successor package tw-animate-css: apps/web/src/app/globals.css:6 uses @import 'tw-animate-css';, and apps/web/package.json declares tw-animate-css@1.4.0. Storybook inherits this via apps/storybook/.storybook/storybook.css which imports ../../web/src/app/globals.css.
  • tailwindcss-animate is a leaf: its only peer dependency is tailwindcss (already declared by the storybook package at 4.2.4). Removing it produced a lockfile diff that is exactly the three expected entries (importer, package resolution, snapshot) and nothing else.

Dynamic, configuration, and tooling references checked

  • CSS: apps/storybook/.storybook/storybook.css and apps/web/src/app/globals.css contain no @plugin/@import for tailwindcss-animate (they use tw-animate-css).
  • Tailwind/PostCSS config: apps/storybook/tailwind.config.js has plugins: []; apps/storybook/postcss.config.mjs only configures @tailwindcss/postcss.
  • Storybook config: apps/storybook/.storybook/main.ts addons are @storybook/addon-docs only; no reference to the plugin.
  • Scripts: apps/storybook/package.json scripts do not reference the package.
  • CI: no workflow references it.
  • Static analysis: knip (apps/web/knip.ts + storybook workspace) reports tailwindcss-animate under "Unused devDependencies", agreeing with the manual search.

Package publication and peer-dependency considerations

  • @kilocode/storybook is a private workspace package ("private": true), not published; there is no published-package contract to preserve.
  • tailwindcss-animate was a regular devDependency, not a peer/optional dependency.
  • It is not a framework plugin, loader, preset, code-generation, or runtime-adapter dependency that would be discovered by convention.

Validation performed

  • pnpm install --no-frozen-lockfile completed; lockfile diff contains only the removal of the three tailwindcss-animate entries (no unrelated changes).
  • pnpm --filter @kilocode/storybook run typecheck (tsgo --noEmit) passed.
  • git diff --check passed; package.json re-parsed as valid JSON.
  • No undeclared transitive dependency usage was exposed: the removed package is a leaf whose sole peer (tailwindcss) remains declared.

Not run (environment constraint): build-storybook / Chromatic (requires the web build and, for some paths, PostgreSQL; this sandbox cannot start postgres/redis and is resource-constrained). Storybook lint is not applicable — the package has no src/ and no lint script.

Uncertainty

  • @storybook/test-runner and @tailwindcss/typography were also flagged by knip as unused in @kilocode/storybook but were intentionally not removed:
    • @storybook/test-runner removal caused pnpm to drop @swc/core@1.15.18 from the graph, which apps/web's @swc/jest relies on as a peer (jest transform would break). This is a pre-existing undeclared-peer situation and is out of scope for this change.
    • @tailwindcss/typography is referenced by @plugin '@tailwindcss/typography' in apps/web/src/app/globals.css, which Storybook imports; removing the Storybook copy risks the Storybook Tailwind build if plugin resolution is relative to the importing CSS. Not high-confidence, so left in place.

The repo migrated to tw-animate-css; no CSS, config, script, or source
file references tailwindcss-animate. It is a leaf package whose only
peer (tailwindcss) is already declared, so removal changes nothing else
in the lockfile.
@kilo-code-bot kilo-code-bot Bot added the janitor Automated dead-code/duplication cleanup label Sep 24, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

This PR removes the unused tailwindcss-animate devDependency from apps/storybook, and verification confirms no CSS, config, script, source, or lockfile consumer references it.

Verification Notes

  • Repository-wide search finds tailwindcss-animate only in apps/web/src/tests/req_sample/anthropic-claude37.log.req.json, a stale historical fixture, not a consumer.
  • Animation utilities are supplied by the successor tw-animate-css, imported in apps/web/src/app/globals.css:6, which apps/storybook/.storybook/storybook.css imports.
  • apps/storybook/tailwind.config.js has plugins: [], and neither postcss.config.mjs nor .storybook/main.ts references the plugin.
  • pnpm-lock.yaml retains no tailwindcss-animate entries after the change; the removed importer/package/snapshot entries are consistent.
Files Reviewed (2 files)
  • apps/storybook/package.json
  • pnpm-lock.yaml (generated)

Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

janitor Automated dead-code/duplication cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant