Skip to content

chore(KNO-14577): update telegraph dependencies - #1055

Merged
kylemcd merged 1 commit into
mainfrom
kyle-kno-14577-update-telegraph-dependencies-in-the-javascript-sdk-monorepo
Jul 31, 2026
Merged

chore(KNO-14577): update telegraph dependencies#1055
kylemcd merged 1 commit into
mainfrom
kyle-kno-14577-update-telegraph-dependencies-in-the-javascript-sdk-monorepo

Conversation

@kylemcd

@kylemcd kylemcd commented Jul 31, 2026

Copy link
Copy Markdown
Member

Description

every @telegraph/* package had a minor release waiting. @telegraph/icon 0.6.0 is the only breaking one: alt and aria-hidden are now mutually exclusive.

  • four example icons passed both alt and aria-hidden — dropped alt, each sits beside visible text
  • two of those had a stale alt="GitHub" sitting next to "Documentation" text
  • turbo's type check skips the examples, where the only break was — ran tsc --noEmit there
  • the lockfile shrinks: layout and style-engine each had two copies, now one
  • the one non-telegraph move is postcss 8.5.23 → 8.5.25, pulled by style-engine@0.4.0
  • version floors across three manifests, the lockfile, and the changeset are mechanical

Bump all @telegraph/* packages to their latest minor releases. Every
package resolves to a single version, and transitive deps move up too
(helpers 0.2.0, menu 0.6.0, truncate 0.3.0, appearance 0.0.15).

@telegraph/icon 0.6.0 makes IconProps an XOR: an icon is either labelled
via alt or explicitly hidden via aria-hidden, never both. Four call sites
in the examples passed both and no longer compiled. Each sits beside a
visible text label, so the icon is decorative — kept aria-hidden and
dropped alt. That also cleared a stale alt="GitHub" on the icon next to
"Documentation" in both files.
@linear-code

linear-code Bot commented Jul 31, 2026

Copy link
Copy Markdown

KNO-14577

@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
javascript-ms-teams-connect-example Ready Ready Preview Jul 31, 2026 1:59pm
javascript-nextjs-example Ready Ready Preview Jul 31, 2026 1:59pm
javascript-slack-connect-example Ready Ready Preview Jul 31, 2026 1:59pm
javascript-slack-kit-example Ready Ready Preview Jul 31, 2026 1:59pm

Request Review

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 344 bytes (0.06%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@knocklabs/react-react-esm 153.86kB 172 bytes (0.11%) ⬆️
@knocklabs/react-react-cjs 131.14kB 172 bytes (0.13%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: @knocklabs/react-react-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
modules/guide/components/Toolbar/styles.inline.js 172 bytes 25.4kB 0.68%
view changes for bundle: @knocklabs/react-react-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
modules/guide/components/Toolbar/styles.inline.mjs 172 bytes 25.48kB 0.68%

@kylemcd
kylemcd marked this pull request as ready for review July 31, 2026 15:25
@kylemcd
kylemcd requested a review from a team as a code owner July 31, 2026 15:25
@kylemcd
kylemcd requested review from andy-knock and meryldakin and removed request for a team July 31, 2026 15:25
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.00%. Comparing base (36e8147) to head (fad82e7).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1055   +/-   ##
=======================================
  Coverage   73.00%   73.00%           
=======================================
  Files         139      139           
  Lines        4342     4342           
  Branches     1275     1275           
=======================================
  Hits         3170     3170           
  Misses       1055     1055           
  Partials      117      117           

@kylemcd
kylemcd merged commit 9dddaf7 into main Jul 31, 2026
13 checks passed
@kylemcd
kylemcd deleted the kyle-kno-14577-update-telegraph-dependencies-in-the-javascript-sdk-monorepo branch July 31, 2026 15:50
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.

2 participants