Skip to content

[chore] Revert ts dep bumps from #5337#5358

Merged
bekossy merged 3 commits into
release/v0.105.2from
revert/pr-5337
Jul 17, 2026
Merged

[chore] Revert ts dep bumps from #5337#5358
bekossy merged 3 commits into
release/v0.105.2from
revert/pr-5337

Conversation

@ashrafchowdury

Copy link
Copy Markdown
Contributor

Context

PR #5337 bumped the TypeScript dependencies across the web workspace. This takes those
bumps back out.

The services/runner half of #5337 (daytona SDK, OTel, undici) was already reverted on
main by the walk-backs at 019f24b and 7a3190a, so what was left is confined to web/.

Changes

Reverts the web dependency bumps from #5337 (merge commit 22900cf): the package
manifests, the lockfile, and the source reformatting that its "run hooks" commit applied
when prettier went from 3.7.4 to 3.9.5.

web/pnpm-lock.yaml is restored verbatim from before the PR rather than regenerated.
This is the part worth knowing: deleting the lockfile and reinstalling does not revert a
dependency bump. A fresh resolve re-reads every caret range and picks whatever is newest
today, which is mostly what the PR installed ("prettier": "^3.7.4" resolves straight
back to 3.9.5). Regenerating reverted only 73 of the 454 package versions #5337
introduced. Restoring the file verbatim reverts all 454.

Two later fixes are deliberately kept rather than reverted:

  • @playwright/test stays pinned at exactly 1.60.0 (a35eb91). Restoring the pre-PR
    ^1.60.0 range would float to 1.61.x and reintroduce the Node 22.15+/24 async loader
    failure that e3bb3e6 fixed.
  • The playwright locator fixes (2f81c12, e3bb3e6) are untouched.

Notes

  • pnpm i against the restored lockfile downgrades cleanly: prettier 3.9.5 to 3.7.4,
    tsx 4.23.1 to 4.22.4, typescript-eslint 8.64.0 to 8.61.0, next 15.5.20 to 15.5.18.
  • Verified none of the 454 package versions [chore] Bump ts deps #5337 introduced remain in the lockfile.
  • turbo lint passes and the prettier hook is a no-op, which confirms the formatting
    revert is consistent with 3.7.4.
  • Heads-up for anyone who already ran pnpm i on the newer tree: reinstall after this
    merges, or your next commit will reformat those files back.

jp-agenta and others added 2 commits July 17, 2026 09:40
This reverts the web dependency bumps from PR #5337 (merge commit
22900cf), restoring the pre-PR versions across the web workspace,
including the source reformatting its "run hooks" commit applied.

The services/runner half of #5337 (daytona SDK, OTel, undici) was
already reverted on main by 019f24b, 7a3190a and the two subsequent
revert commits, so this change is confined to web/.

web/pnpm-lock.yaml is restored verbatim from before the PR rather than
regenerated. A fresh resolve would re-float every caret-ranged dep back
to the versions the PR installed (prettier ^3.7.4 resolves to 3.9.5
today), which reverts the manifests but not the actual dependency tree.

@playwright/test is deliberately left pinned at 1.60.0 (a35eb91) rather
than restored to the pre-PR "^1.60.0" range, which would float to 1.61.x
and reintroduce the Node 22.15+/24 async loader failure fixed in
e3bb3e6. The later playwright locator fixes (2f81c12, e3bb3e6) are
likewise untouched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 17, 2026
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Jul 17, 2026 12:44pm

Request Review

@dosubot dosubot Bot added the dependencies Dependencies label Jul 17, 2026
@ashrafchowdury
ashrafchowdury requested a review from ardaerzin July 17, 2026 10:26
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0a234def-9741-4502-b1b2-6f7f805ffa3d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

This PR refreshes web workspace dependency declarations, changes internal package references to the workspace: protocol, adjusts Next.js and pnpm overrides, and reformats equivalent TypeScript unions, assertions, and annotations across frontend packages.

Changes

Web dependency and workspace alignment

Layer / File(s) Summary
Dependency and workspace manifest alignment
web/*/package.json, web/packages/*/package.json, web/pnpm-workspace.yaml
Internal package links use workspace: references, dependency versions are refreshed, and Next.js/pnpm override configuration is updated.
OSS application type formatting
web/oss/src/**
Equivalent TypeScript unions and assertions are reformatted across OSS components, state, selectors, and utilities.
Annotation and entity type formatting
web/packages/agenta-annotation/src/**, web/packages/agenta-entities/src/**
Annotation column construction and entity-related type expressions are reformatted without runtime logic changes.
Entity UI type and assertion formatting
web/packages/agenta-entity-ui/src/**
Schema controls, selection adapters, modal types, and view types receive equivalent multiline type expressions.
Playground, shared, and UI formatting
web/packages/agenta-playground*/src/**, web/packages/agenta-shared/src/**, web/packages/agenta-ui/src/**
Local annotations and exported unions are reformatted while preserving their effective types and behavior.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 92.31% which is sufficient. The required threshold is 60.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title matches the main change: reverting TypeScript dependency bumps from PR #5337.
Description check ✅ Passed The description is directly about reverting the web workspace dependency bumps and related formatting changes from #5337.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert/pr-5337

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Railway Preview Environment

Preview URL https://gateway-production-be4c.up.railway.app/w
Project agenta-oss-pr-5358
Image tag pr-5358-c74fc43
Status Deployed
Railway logs Open logs
Workflow logs View workflow run
Updated at 2026-07-17T12:54:28.934Z

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (2)
web/packages/agenta-entities/src/runnable/utils.ts (1)

829-831: 📐 Maintainability & Code Quality | 🔵 Trivial

Run the required frontend lint-fix before committing.

Please run pnpm lint-fix from the web folder; the supplied validation mentions turbo lint, but does not confirm this required command.

As per coding guidelines, frontend changes require running pnpm lint-fix within the web folder before committing.

Source: Coding guidelines

web/oss/package.json (1)

22-134: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Move @agenta/web-tests to devDependencies. It’s only imported from the Playwright test suite under web/oss/tests/playwright/acceptance, so it doesn’t belong in the runtime dependency set; web/ee/package.json already places it in devDependencies.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 440d2323-1642-4b12-96c9-fc1b0e42a7ef

📥 Commits

Reviewing files that changed from the base of the PR and between 327232b and 06dff7a.

⛔ Files ignored due to path filters (1)
  • web/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (108)
  • web/ee/package.json
  • web/oss/package.json
  • web/oss/src/components/AgentChatSlice/AgentConversation.tsx
  • web/oss/src/components/AgentChatSlice/assets/trace.ts
  • web/oss/src/components/EvalRunDetails/atoms/mutations/editEvaluation.ts
  • web/oss/src/components/EvalRunDetails/atoms/runMetrics.ts
  • web/oss/src/components/EvalRunDetails/atoms/table/types.ts
  • web/oss/src/components/EvalRunDetails/components/views/OverviewView/hooks/useRunMetricData.ts
  • web/oss/src/components/EvalRunDetails/utils/buildAnnotationMetricData.ts
  • web/oss/src/components/EvaluationRunsTablePOC/components/EvaluationRunsTable/index.tsx
  • web/oss/src/components/Evaluators/components/ConfigureEvaluator/index.tsx
  • web/oss/src/components/Filters/Filters.tsx
  • web/oss/src/components/InfiniteVirtualTable/atoms/columnVisibility.ts
  • web/oss/src/components/InfiniteVirtualTable/columns/createStandardColumns.tsx
  • web/oss/src/components/InfiniteVirtualTable/hooks/useHeaderViewportVisibility.ts
  • web/oss/src/components/Playground/Components/Modals/RefinePromptModal/hooks/useRefinePrompt.ts
  • web/oss/src/components/Playground/Components/PlaygroundFocusDrawerAdapter/index.tsx
  • web/oss/src/components/Playground/Playground.tsx
  • web/oss/src/components/Playground/PlaygroundTokenPath/atoms.ts
  • web/oss/src/components/SharedDrawers/AnnotateDrawer/assets/hooks/useEvaluatorSchemas.ts
  • web/oss/src/components/SharedDrawers/AnnotateDrawer/assets/transforms.ts
  • web/oss/src/components/SharedDrawers/TraceDrawer/store/traceDrawerStore.ts
  • web/oss/src/components/Sidebar/engine/SidebarMenu.tsx
  • web/oss/src/components/Webhooks/WebhookDrawer.tsx
  • web/oss/src/components/Webhooks/assets/types.ts
  • web/oss/src/components/WorkflowRevisionDrawerWrapper/index.tsx
  • web/oss/src/components/pages/observability/assets/filters/referenceUtils.ts
  • web/oss/src/lib/helpers/analytics/hooks/useSurvey.ts
  • web/oss/src/lib/hooks/useAnnotations/types/index.ts
  • web/oss/src/lib/hooks/usePreviewEvaluations/assets/previewRunBatcher.ts
  • web/oss/src/services/app-selector/api/index.ts
  • web/oss/src/state/appCreation/status.ts
  • web/oss/src/state/newObservability/atoms/controls.ts
  • web/oss/src/state/workflow/destinations.ts
  • web/package.json
  • web/packages/agenta-annotation-ui/package.json
  • web/packages/agenta-annotation/package.json
  • web/packages/agenta-annotation/src/state/controllers/annotationSessionController.ts
  • web/packages/agenta-api-client/package.json
  • web/packages/agenta-entities/package.json
  • web/packages/agenta-entities/src/loadable/controller.ts
  • web/packages/agenta-entities/src/runnable/evaluatorTransforms.ts
  • web/packages/agenta-entities/src/runnable/types.ts
  • web/packages/agenta-entities/src/runnable/utils.ts
  • web/packages/agenta-entities/src/shared/execution/schema.ts
  • web/packages/agenta-entities/src/shared/openapi/schemaUtils.ts
  • web/packages/agenta-entities/src/shared/relations/extendWithRelations.ts
  • web/packages/agenta-entities/src/shared/tableTypes.ts
  • web/packages/agenta-entities/src/testcase/state/molecule.ts
  • web/packages/agenta-entities/src/workflow/api/createFromTemplate.ts
  • web/packages/agenta-entities/src/workflow/core/evaluatorResolution.ts
  • web/packages/agenta-entities/src/workflow/snapshotAdapter.ts
  • web/packages/agenta-entities/src/workflow/state/appUtils.ts
  • web/packages/agenta-entities/src/workflow/state/commit.ts
  • web/packages/agenta-entities/src/workflow/state/molecule.ts
  • web/packages/agenta-entities/src/workflow/state/runnableSetup.ts
  • web/packages/agenta-entities/src/workflow/state/store.ts
  • web/packages/agenta-entity-ui/package.json
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/FeedbackConfigurationControl.tsx
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/PromptSchemaControl.tsx
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/ToolItemControl.tsx
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/ToolSelectorPopover.tsx
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/agentTemplate/useAgentTools.ts
  • web/packages/agenta-entity-ui/src/DrillInView/SchemaControls/agentTemplate/useModelHarness.tsx
  • web/packages/agenta-entity-ui/src/DrillInView/components/PlaygroundConfigSection.tsx
  • web/packages/agenta-entity-ui/src/adapters/variantAdapters.ts
  • web/packages/agenta-entity-ui/src/modals/actions/types.ts
  • web/packages/agenta-entity-ui/src/modals/types.ts
  • web/packages/agenta-entity-ui/src/selection/adapters/createAdapterFromRelations.ts
  • web/packages/agenta-entity-ui/src/selection/adapters/createLevelFromRelation.ts
  • web/packages/agenta-entity-ui/src/selection/components/UnifiedEntityPicker/types.ts
  • web/packages/agenta-entity-ui/src/selection/hooks/modes/autoSelectLatestChild.ts
  • web/packages/agenta-entity-ui/src/view-types/viewTypes.ts
  • web/packages/agenta-playground-ui/package.json
  • web/packages/agenta-playground-ui/src/components/ExecutionItemComparisonView/GenerationComparisonCompletionOutput/index.tsx
  • web/packages/agenta-playground-ui/src/components/ExecutionItems/assets/ChatTurnView/index.tsx
  • web/packages/agenta-playground-ui/src/components/ExecutionItems/assets/ExecutionRow/ComparisonLayout.tsx
  • web/packages/agenta-playground-ui/src/components/ExecutionItems/assets/ExecutionRow/SingleLayout.tsx
  • web/packages/agenta-playground-ui/src/components/PlaygroundOutputs/index.tsx
  • web/packages/agenta-playground-ui/src/components/WorkflowRevisionDrawer/store.ts
  • web/packages/agenta-playground-ui/src/components/shared/EvaluatorFieldGrid/index.tsx
  • web/packages/agenta-playground-ui/src/components/shared/NodeResultCard/index.tsx
  • web/packages/agenta-playground/package.json
  • web/packages/agenta-playground/src/state/atoms/modeOverride.ts
  • web/packages/agenta-playground/src/state/controllers/playgroundController.ts
  • web/packages/agenta-playground/src/state/controllers/playgroundSnapshotController.ts
  • web/packages/agenta-playground/src/state/controllers/traceRefResolution.ts
  • web/packages/agenta-playground/src/state/execution/agentRequest.ts
  • web/packages/agenta-playground/src/state/execution/executionItems.ts
  • web/packages/agenta-playground/src/state/execution/executionRunner.ts
  • web/packages/agenta-playground/src/state/helpers/entityInputContract.ts
  • web/packages/agenta-sdk/package.json
  • web/packages/agenta-shared/package.json
  • web/packages/agenta-shared/src/utils/chatMessage.ts
  • web/packages/agenta-shared/src/utils/elicitation.ts
  • web/packages/agenta-shared/src/utils/mappingUtils.ts
  • web/packages/agenta-shared/src/utils/mustache/parser.ts
  • web/packages/agenta-shared/src/utils/schemaOptions.ts
  • web/packages/agenta-ui/package.json
  • web/packages/agenta-ui/src/ChatMessage/components/ChatInputs.tsx
  • web/packages/agenta-ui/src/Editor/form/nodes/ObjectNode.tsx
  • web/packages/agenta-ui/src/InfiniteVirtualTable/atoms/columnVisibility.ts
  • web/packages/agenta-ui/src/InfiniteVirtualTable/columns/createStandardColumns.tsx
  • web/packages/agenta-ui/src/InfiniteVirtualTable/hooks/useHeaderViewportVisibility.ts
  • web/packages/agenta-ui/src/drill-in/coreTypes.ts
  • web/packages/agenta-ui/src/utils/styles.ts
  • web/pnpm-workspace.yaml
  • web/tests/package.json
💤 Files with no reviewable changes (1)
  • web/pnpm-workspace.yaml

@bekossy
bekossy changed the base branch from main to release/v0.105.2 July 17, 2026 11:09
The revert of PR #5337 removed the `overrides` block from
web/pnpm-workspace.yaml along with the ts dep bumps, without mentioning
it. Restoring pnpm-lock.yaml verbatim to the pre-PR state re-floated
three packages below the floors those overrides enforced: vite (8.0.12
< ^8.0.16), postcss bundled inside next (8.4.31 < >=8.5.10), and
minimatch via an old typescript-eslint dev dependency (9.0.3 <
>=9.0.7). Re-adds the overrides and re-locks just enough to satisfy
them, without re-floating the caret-ranged deps (prettier etc.) the
revert intentionally pinned.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 17, 2026
@bekossy
bekossy merged commit 457fef1 into release/v0.105.2 Jul 17, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependencies lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants