Skip to content

(janitor/dependencies) remove unused itty-time from gastown and wasteland - #5927

Merged
eshurakov merged 1 commit into
mainfrom
janitor/dependencies-remove-itty-time
Sep 7, 2026
Merged

(janitor/dependencies) remove unused itty-time from gastown and wasteland#5927
eshurakov merged 1 commit into
mainfrom
janitor/dependencies-remove-itty-time

Conversation

@kilo-code-bot

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

Copy link
Copy Markdown
Contributor

Dependencies removed

  • itty-time from services/gastown (was 1.0.6)
  • itty-time from services/wasteland (was ^1.0.6)

Both are private, non-published Cloudflare Worker services, so removing an unused runtime
dependency carries no published-package contract impact.

Evidence each was unused

itty-time is a small standalone time-parsing/formatting utility from the itty ecosystem.
It has no imports anywhere in the repository:

  • A full-tree search (excluding node_modules, .git, and pnpm-lock.yaml) for
    itty-time matches only the two package.json declarations — zero source, config,
    test, or script references.
  • No import ... from 'itty-time', require('itty-time'), dynamic import(), or
    import type references exist in either service (or anywhere else in the monorepo).
  • It is not referenced by wrangler.jsonc/wrangler.toml, vitest.config.ts,
    tsconfig.json, or any oxlint/build configuration for either service.
  • It has no side-effectful entry point and is not discovered by any naming convention.

This is consistent with a dependency left behind after the code that used it was removed.

Dynamic, configuration, and tooling references checked

  • No dynamic imports or require calls.
  • Not a framework/compiler plugin, preset, loader, runtime adapter, or code-generation
    tool (contrast with e.g. @hono/trpc-server, @sentry/cloudflare, or wrangler).
  • Not listed in the pnpm workspace catalog.
  • Package scripts for both services (typecheck, lint, test) were verified to still
    load and run after removal.

Package publication and peer-dependency considerations

  • cloudflare-gastown and cloudflare-wasteland are both "private": true; neither is
    published to npm, so no peer/optional/contract obligations apply.
  • itty-time is a leaf package in the lockfile (itty-time@1.0.6: {} in snapshots) with
    no dependents and no peers of its own, so its removal cannot unmask any undeclared
    transitive dependency or introduce a peer-resolution gap.
  • The pre-existing pnpm peers check warning is unchanged by this removal (leaf package,
    no peers).

Validation performed

  • pnpm install --prefer-offline completed cleanly; lockfile diff is limited to the three
    itty-time entries (two importers + one package/snapshot pair), 13 deletions, no
    unrelated changes.
  • pnpm --filter cloudflare-gastown typecheck — passed (0 errors).
  • pnpm --filter cloudflare-wasteland typecheck — passed (0 errors).
  • pnpm --filter cloudflare-gastown lint — 0 warnings, 0 errors.
  • pnpm --filter cloudflare-wasteland lint — 0 warnings, 0 errors.
  • pnpm --filter cloudflare-gastown test — 335 passed (22 files).
  • pnpm --filter cloudflare-wasteland test — 126 passed (12 files).

Note: these service test suites are Durable Object / unit tests and do not require
PostgreSQL or Redis, which are unavailable in this sandbox; no DB-backed checks were
skipped as a result of the change.

Uncertainty

  • None material. The only caveat is that the runtime container could not be exercised
    end-to-end in this environment (no wrangler deploy), but since itty-time is never
    imported, the produced Worker bundle is unaffected by its removal.

@kilo-code-bot kilo-code-bot Bot added the janitor Automated dead-code/duplication cleanup label Sep 7, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • pnpm-lock.yaml
  • services/gastown/package.json
  • services/wasteland/package.json

Reviewed by grok-4.6 · Input: 65.7K · Output: 4.5K · Cached: 214.8K

Review guidance: REVIEW.md from base branch main

@eshurakov
eshurakov merged commit 71efe3b into main Sep 7, 2026
24 checks passed
@eshurakov
eshurakov deleted the janitor/dependencies-remove-itty-time branch September 7, 2026 13:36
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