Skip to content

Clarify bundled third-party dependencies#395

Merged
BorisTyshkevich merged 1 commit into
mainfrom
docs/clarify-bundled-dependencies
Jul 23, 2026
Merged

Clarify bundled third-party dependencies#395
BorisTyshkevich merged 1 commit into
mainfrom
docs/clarify-bundled-dependencies

Conversation

@BorisTyshkevich

Copy link
Copy Markdown
Collaborator

What changed

  • Replace the misleading “zero third-party requests” marketing claim with precise language: runtime dependencies are bundled and no third-party CDN assets are loaded.
  • Correct the documented runtime dependency count to seven and include date-fns.
  • Add the missing license notices for chartjs-adapter-date-fns, date-fns, and marked.

Why

The application has third-party runtime dependencies and contacts the configured OAuth IdP. The previous wording conflated bundled dependencies with network requests and omitted several packages from the notices.

Verification

  • npm test — 167 files / 5,074 tests; 100% statements and lines
  • npm run build
  • Verified the new notices are embedded in dist/sql.html
  • git diff --check

@BorisTyshkevich
BorisTyshkevich merged commit 42b58bc into main Jul 23, 2026
9 checks passed
@BorisTyshkevich
BorisTyshkevich deleted the docs/clarify-bundled-dependencies branch July 23, 2026 04:40
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