Skip to content

feat: expand KNOWN_RUNTIMES allowlist in dependency checker#22

Merged
theDakshJaitly merged 1 commit intotheDakshJaitly:mainfrom
mvanhorn:osc/2-expand-known-runtimes
Apr 7, 2026
Merged

feat: expand KNOWN_RUNTIMES allowlist in dependency checker#22
theDakshJaitly merged 1 commit intotheDakshJaitly:mainfrom
mvanhorn:osc/2-expand-known-runtimes

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

@mvanhorn mvanhorn commented Apr 7, 2026

Summary

Expands the KNOWN_RUNTIMES set in the dependency checker to cover common non-package terms that appear in stack documentation but get false DEPENDENCY_MISSING warnings.

Added terms

  • Languages: Swift, Kotlin, Elixir, Erlang, PHP, .NET/C#
  • Databases: DynamoDB, Cassandra, Neo4j, Supabase, Neon
  • Cloud providers: Vercel, Netlify, Railway, Fly.io, Render, AWS/GCP/Azure/Cloudflare
  • AWS services: S3, EC2, Lambda, ECS, Fargate
  • Protocols: REST, GraphQL, gRPC, WebSocket, OAuth/JWT/SAML/OIDC
  • CSS/Build: Tailwind CSS, Bootstrap, Sass, Webpack, Vite, esbuild, Turbopack
  • Infrastructure: Git, GitHub, GitLab, Nginx, Apache, Caddy, Linux, WASM

Fixes #2

This contribution was developed with AI assistance (Claude Code).

Add common non-package terms that appear in stack documentation:
- Languages: Swift, Kotlin, Elixir, PHP, .NET/C#
- Databases: DynamoDB, Cassandra, Neo4j, Supabase, Neon
- Cloud: Vercel, Netlify, Railway, AWS services (S3, EC2, Lambda)
- Protocols: REST, GraphQL, gRPC, WebSocket, OAuth, JWT
- CSS/Build: Tailwind, Bootstrap, Webpack, Vite, esbuild
- Infra: Git, GitHub, Nginx, Linux, WASM

Prevents false DEPENDENCY_MISSING warnings for these terms.

Fixes theDakshJaitly#2
@theDakshJaitly theDakshJaitly self-requested a review April 7, 2026 14:42
@theDakshJaitly
Copy link
Copy Markdown
Owner

Looks good — merging! A couple things we'll clean up on our end:

  1. Some entries are actual npm packagestailwindcss, webpack, vite, esbuild, sass, postcss, bootstrap, rollup, parcel are all installable via npm. Having them in the allowlist means we won't catch them if they're claimed but actually missing from package.json. We'll trim those out.

  2. Variable namingKNOWN_RUNTIMES is getting stretched now that it covers CSS frameworks, build tools, and protocols. We'll likely rename it to something like KNOWN_NON_PACKAGES.

Thanks for the contribution!

@theDakshJaitly theDakshJaitly merged commit 9ce1dd7 into theDakshJaitly:main Apr 7, 2026
3 checks passed
@mvanhorn
Copy link
Copy Markdown
Contributor Author

mvanhorn commented Apr 8, 2026

Thanks for the merge!

@theDakshJaitly
Copy link
Copy Markdown
Owner

appreciate it, thanks for contribution!

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.

Expand KNOWN_RUNTIMES allowlist in dependency checker

2 participants