Skip to content

chore(minting): drop unused LICENSE_DEFAULT_TTL_DAYS#536

Merged
blove merged 1 commit into
mainfrom
claude/drop-default-ttl-days
May 25, 2026
Merged

chore(minting): drop unused LICENSE_DEFAULT_TTL_DAYS#536
blove merged 1 commit into
mainfrom
claude/drop-default-ttl-days

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 25, 2026

Summary

Under subscriptions, every license's `expiresAt` comes from `item.current_period_end`. The fallback fixed TTL has no callers, so the env var and the HandlerDeps field were dead code.

Test plan

  • minting-service tests pass
  • Drop LICENSE_DEFAULT_TTL_DAYS from the Vercel project env (operational, after merge)

🤖 Generated with Claude Code

Under subscriptions, every license's expiresAt comes from the
subscription's item.current_period_end. There is no fallback fixed
TTL anywhere in the codepath, so the env var and the deps field were
dead code.

- env.ts: drop LICENSE_DEFAULT_TTL_DAYS from Env type + loadEnv
- handlers.ts: drop defaultTtlDays from HandlerDeps
- stripe-webhook.ts: drop defaultTtlDays from the deps wiring
- env.spec / handlers.spec: drop the assertion / fixture

Operational follow-up: LICENSE_DEFAULT_TTL_DAYS can be removed from
the Vercel minting-service project env vars (no runtime effect since
nothing reads it).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment May 25, 2026 3:01pm

Request Review

@blove blove merged commit 88b070e into main May 25, 2026
22 checks passed
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