chore(skill): Add skill to summarize framework updates#21361
Open
s1gr1d wants to merge 7 commits into
Open
Conversation
s1gr1d
requested review from
a team,
chargome and
nicohrubec
and removed request for
a team
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| ) | ||
| args = parser.parse_args() | ||
|
|
||
| os.makedirs(os.path.dirname(args.out), exist_ok=True) |
Contributor
There was a problem hiding this comment.
Bug: The script crashes if the --out argument is a bare filename, as os.makedirs is called with an empty string.
Severity: LOW
Suggested Fix
Before creating the directory, ensure the path is absolute to correctly handle bare filenames. For example, use os.path.dirname(os.path.abspath(args.out)) to get the directory name. This will ensure a valid directory path is passed to os.makedirs.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: .agents/skills/track-framework-updates/scripts/collect_updates.py#L100
Potential issue: The script `collect_updates.py` will crash if the `--out` argument is
provided with a bare filename (e.g., `output.json`) instead of a full path. This is
because `os.path.dirname()` on a bare filename returns an empty string, and the
subsequent call `os.makedirs("", exist_ok=True)` raises a `FileNotFoundError`. While the
documented default usage is safe because it constructs a full path, the script's
interface is fragile and will fail under realistic alternative usage patterns where a
user or another script provides just a filename.
Did we get this right? 👍 / 👎 to inform future reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a skill to summarize relevant framework updates from the last 7 days.
It looks at releases, RSS feeds, blogs and GitHub discussions. The skill includes some scripts that take care of data fetching (the deterministic parts). The collected data is then brought to the skill, which handles checking for relevance (the non-deterministic parts).
Folder structure
Mermaid Flowchart
Example Summary (v1)
Framework Updates Digest — week of 2026-06-08
Window: last 7 days · generated 2026-06-08T09:36:33Z
TL;DR
@sentry/angularsupport matrix.$app/envmodule rename, adapters moving torolldown, TS 6 minimum,query.livenow over SSE.4.4.7and3.21.7).cacheHandlertracing.Backlog candidates
$app/envmodule rename (with$app/environmentreinstated as an alias), adapters migrating rollup→rolldown, TS 6 minimum, andquery.livemoving to SSE → assess SDK compatibility early since the major is forming now. (kit 3.0.0-next.0, next.1, adapter-node 6)@sentry/nuxtcompatibility, bump the E2E matrix. A user already reports a Cloudflare deploy regression after 4.4.7. (v4.4.7, v3.21.7, advisories)cacheHandler(s)when using adapters → verify our server wrapping doesn't rely on the deprecated methods and check whether staged App Shell rendering shifts pageload/navigation span boundaries. (canary.40, canary.41, canary.44)Client-Side
Angular (@sentry/angular)
Releases
Interesting links
React (@sentry/react)
Releases
Vue (@sentry/vue)
Releases
Svelte (@sentry/svelte)
Releases
Interesting links
Ember (@sentry/ember)
Interesting links
import Ember from "ember"' to Recommended — Advance RFC #1003"Deprecateimport Ember from 'ember';"` to Stage Recommended emberjs/rfcs#1110"Deprecating Mixin Support"to Stage Ready for Release emberjs/rfcs#1143"First-Class Component Templates"to Stage Recommended emberjs/rfcs#1059Server-Side
Hono (@sentry/hono)
Interesting links
c.req.json()returnanyinstead ofunknown? — https://github.com/orgs/honojs/discussions/4979Nitro (@sentry/nitro)
Releases
defaultPreset. Preset/runtime changes can affect@sentry/nuxt+@sentry/nitroserver setup; keep tracking the v3 beta line.NestJS (@sentry/nestjs)
Releases
Effect (@sentry/effect)
Releases
Meta-Framework
Next.js (@sentry/nextjs)
Releases
cacheHandler(s)with adapters — verify our server wrapping is unaffected.Interesting links
router.push()does not work on SSG pages loaded with existing searchParams — router.push() does not work on SSG pages when page is loaded with existing searchParams vercel/next.js#94530Nuxt (@sentry/nuxt)
Releases
Interesting links
SvelteKit (@sentry/sveltekit)
Releases
$app/environmentas an alias for$app/env— confirms an env-module rename our SDK may import.query.livenow uses SSE; check our request/streaming instrumentation on 2.x.Interesting links
React Router / Remix (@sentry/react-router · @sentry/remix)
Releases
Interesting links
Astro (@sentry/astro)
Releases
TanStack Start (@sentry/tanstackstart · @sentry/tanstackstart-react)
Releases
Interesting links
No notable upstream activity this week for: Solid, SolidStart, Gatsby, Elysia.
Example Summary (v2)
Framework Updates Digest — week of 2026-06-08
Window: last 7 days · generated 2026-06-08T11:30:00Z
TL;DR
injectAsync, signal debouncing, SSR resource caching,ChangeDetectionStrategy.Eager)inputValidator()in favor of newvalidator()API for server functions/middlewareBacklog candidates
injectAsync,provideWebMcpTools,ChangeDetectionStrategy.Eager, signal debouncing, and SSR resource caching → Investigate whether these new APIs need instrumentation or affect existing hooks. (v22.0.0)@sveltejs/kit/node/polyfills, requires Vite 8/Node 22+, removes deprecated CSRFcheckOrigin, changes cookie path default → Start planning@sentry/sveltekitmigration for v3 compatibility. (@sveltejs/kit@3.0.0-next.0)inputValidator()forvalidator()in server functions and middleware → Check if SDK wraps or references this API. (#7566)Client-Side
Angular (@sentry/angular)
Releases
injectAsync,provideWebMcpTools,ChangeDetectionStrategy.Eager, signal debouncing, SSR resource caching. Many new instrumentation-relevant primitives.Interesting links
React (@sentry/react)
Releases
Vue (@sentry/vue)
Releases
Svelte (@sentry/svelte)
Releases
Interesting links
Server-Side
Hono (@sentry/hono)
Releases
Nitro (@sentry/nitro)
Releases
defaultPresetconfig option. Build/deploy surface extensions.NestJS (@sentry/nestjs)
Releases
Effect (@sentry/effect)
Releases
$match, schema refinement; no SDK impact expected.Tool.EmptyParams; no SDK impact expected.Meta-Framework
Next.js (@sentry/nextjs)
Releases
await instrumentation in RouteModule.prepare— directly relevant to instrumentation hooks.Nuxt (@sentry/nuxt)
Releases
navigateTo,reloadNuxtApp,NuxtLink).getCachedDatare-run after initial fetch.SvelteKit (@sentry/sveltekit)
Releases
query.live, env.d.ts Windows path fix.React Router / Remix (@sentry/react-router, @sentry/remix)
Releases
Interesting links
Astro (@sentry/astro)
Releases
TanStack Start (@sentry/tanstackstart, @sentry/tanstackstart-react)
Releases
validator()as canonical server function/middleware validator; deprecatesinputValidator(). Public API change.Interesting links
Example Summary (v2 with more RSS feeds)
Framework Updates Digest — week of Jun 3, 2026
Window: last 7 days · generated 2026-06-10T15:10+02:00
TL;DR
src/fetch.tsreplacessrc/app.ts),getFetchState()now public, streaming rendering replaces queued rendering —@sentry/astroneeds compatibility review.deltaproperty removed from non-popstatenavigation events (SDK hooks into this); Cloudflare adapter removesplatform.context→platform.ctx— both are breaking for@sentry/sveltekiton SK3.@sentry/angularneeds compatibility audit.catchError/retrystabilised (droppingunstable_prefix),export const prefetchstable, newcacheHandlertracing hooks that could overlap with SDK instrumentation.@sentry/ember.Set-Cookiemerging; trace-context header extraction in those adapters now behaves correctly.Backlog candidates
src/fetch.tsand exposinggetFetchState()for Hono middleware → investigate whether server-side request isolation and trace propagation need updating for the new routing architecture. (release)deltafrom all navigation events exceptpopstate→ audit navigation instrumentation and add a guard before readingevent.delta. (release)platform.contextin favour ofplatform.ctx→ audit and update any Cloudflare request isolation code that readsplatform.context. (release)@sentry/angulartransaction creation or error capture. (blog)cacheHandlerandcacheHandlerswhen using adapters — framework-level cache telemetry → investigate integration or deduplication with SDK-generated cache spans. (canary.40)catchErrorandretryare being stabilised (removingunstable_prefix) as public error-handling APIs → evaluate whether SDK should instrument these for error capture. (canary.47)@sentry/tanstackstartshould expose OTel-compatible hooks. (discussion)@sentry/emberroute transaction instrumentation. (RFC)Client-Side
Angular (
@sentry/angular)Releases
Interesting links
Ember (
@sentry/ember)No releases this week.
Interesting links
Svelte (
@sentry/svelte)Releases
$state.snapshot. Internal fixes only; no SDK impact expected.Interesting links
Vue (
@sentry/vue)Releases
minorbranch CHANGELOG for router, compiler, or lifecycle changes relevant to@sentry/vue, especially Vapor mode.Interesting links
Server-Side
Effect (
@sentry/effect)Releases
Tool.EmptyParamssupport in@effect/ai, shard group fixes in cluster. No SDK instrumentation surfaces changed.Interesting links
Hono (
@sentry/hono)Releases
X-Forwarded-For) — only the last value reached the app. SDK trace-context extraction was silently broken; this fix corrects it.Set-Cookieresponse headers into one comma-separated value — response header shape now correct.NestJS (
@sentry/nestjs)Releases
platform-fastifyremoves trailing pathname slash (changes route matching; SDK route span names in Fastify adapter may differ); SSE close listener ordering fix.Nitro (
@sentry/nitro)Releases
defaultPresetconfig option to customise the fallback deployment preset. Could represent an unrecognised deployment target; monitor for new adapters.Meta-Framework
Astro (
@sentry/astro)Releases
src/fetch.tsreplacessrc/app.tsas the server entrypoint; request flow through SDK middleware hooks changes.getFetchState()exposed fromastro/honoas public API for Hono middleware to access per-request state — new instrumentation surface.state.provide(),state.resolve(),state.finalizeAll(), andApp.Providersfrom the public advanced routing API.context.loggeralways available in API routes.astro:transitionslifecycle event constants and helpers (TRANSITION_BEFORE_PREPARATION,isTransitionBeforePreparationEvent,createAnimationScope, etc.) — SDK navigation tracing relying on these events must migrate to string event names.Astro.request.urlnot reflectingX-Forwarded-Proto/X-Forwarded-Host— corrects the request object the SDK reads for URL-based span attributes.Astro.routePatterncasing, dynamic route fixes. No SDK API surface changes.@astrojs/cloudflare@13.7.0,@astrojs/mdx@6.0.x,@astrojs/markdown-satteri@0.x— No SDK impact expected.Next.js (
@sentry/nextjs)Releases
force-runtime→allow-runtime.export const prefetchas a public routing API.catchErrorandretry(dropunstable_prefix) — new public error-handling APIs.unstable_instantagent hints; stabiliseunstable_instant.experimental.useNodeStreamsflag (streaming now always on).cacheHandlerandcacheHandlerswhen using adapters — framework telemetry for the data cache.Interesting links
Nuxt (
@sentry/nuxt)Releases
findPathtype fix. No SDK impact expected.Interesting links
React Router / Remix (
@sentry/react-router,@sentry/remix)Releases
Interesting links
SvelteKit (
@sentry/sveltekit)Releases
deltafrom all navigation events exceptpopstate— SDK navigation span creation will break if it readsevent.delta.@sveltejs/vite-plugin-sveltev7 and Vite 8 — build plugin major bumps.@sveltejs/kit/node/polyfills, remove deprecated CSRFcheckOrigin, deprecateResponsehelpers.platform.contextin favour ofplatform.ctx— breaks any SDK code that readsplatform.contextfor Cloudflare request isolation.@cloudflare/workers-types4.20260219.0, minimum wrangler^4.67.0.es2022— may affect SDK edge bundle compatibility.Fileobjects; fix: server components not bundled when SSR is off per route.$app/envmodule).Interesting links
TanStack Start (
@sentry/tanstackstart,@sentry/tanstackstart-react)Releases
validator()as canonical server function middleware validator, deprecatesinputValidator(). Patch fixes: scroll restoration, search params.Interesting links