langchain-ai/openwork
1
0
Fork 0
mirror of https://github.com/langchain-ai/openwork.git synced 2026-07-21 01:55:30 -04:00
Code Issues 17 Packages Projects Releases 10 Wiki Activity

[PR #21] [CLOSED] Fix linter errors in core libraries and App #34

New Issue
Closed
opened 2026-02-16 09:17:51 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 09:17:51 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/openwork/pull/21
Author: @bwhiting2356
Created: 1/17/2026
Status: ❌ Closed

Base: lint-fix-3-feature-components ← Head: lint-fix-4-core-libs

📝 Commits (10+)

  • 2be0404 Fix linter errors in backend & main process
  • ea0c272 Fix linter errors in shadcn/ui components
  • 539a1f3 Fix linter errors in backend & main process
  • d6023b8 Fix linter errors in feature components
  • 705dd42 Extract selectWorkspaceFolder to separate utility file
  • b0ae813 Fix ModelSwitcher to derive selectedProviderId instead of setState in effect
  • fc9c803 Revert "Fix ModelSwitcher to derive selectedProviderId instead of setState in effect"
  • 45f6b8b Add TODO comments for eslint-disable directives
  • 6a2ad9d Fix linter errors in core libraries and App
  • ae51aa2 Refactor thread-context into multiple files

📊 Changes

50 files changed (+1111 additions, -1008 deletions)

View changed files

📝 bin/cli.js (+5 -1)
📝 electron.vite.config.ts (+2 -2)
📝 eslint.config.mjs (+23 -17)
📝 src/main/agent/runtime.ts (+3 -1)
📝 src/main/checkpointer/sqljs-saver.ts (+9 -1)
📝 src/main/ipc/agent.ts (+1 -4)
📝 src/main/ipc/threads.ts (+5 -5)
📝 src/main/services/title-generator.ts (+8 -8)
📝 src/main/storage.ts (+1 -1)
📝 src/preload/index.d.ts (+8 -2)
📝 src/preload/index.ts (+11 -3)
📝 src/renderer/src/App.tsx (+52 -52)
📝 src/renderer/src/components/chat/ApiKeyDialog.tsx (+13 -18)
📝 src/renderer/src/components/chat/ChatContainer.tsx (+16 -9)
📝 src/renderer/src/components/chat/ChatTodos.tsx (+2 -2)
📝 src/renderer/src/components/chat/ContextUsageIndicator.tsx (+25 -24)
📝 src/renderer/src/components/chat/MessageBubble.tsx (+32 -38)
📝 src/renderer/src/components/chat/ModelSwitcher.tsx (+34 -35)
📝 src/renderer/src/components/chat/StreamingMarkdown.tsx (+1 -3)
📝 src/renderer/src/components/chat/ToolCallRenderer.tsx (+115 -66)

...and 30 more files

📄 Description

Part 4/4 of linter cleanup. Fixes all lint errors in core libraries and App component.

Stack: #18 → #19 → #20 → #21

Changes

  • Refactor thread-context into multiple files to satisfy react-refresh rules
  • Add missing return type annotations
  • Fix prettier formatting issues

✅ After this PR merges, the codebase will have zero linter errors.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/openwork/pull/21 **Author:** [@bwhiting2356](https://github.com/bwhiting2356) **Created:** 1/17/2026 **Status:** ❌ Closed **Base:** `lint-fix-3-feature-components` ← **Head:** `lint-fix-4-core-libs` --- ### 📝 Commits (10+) - [`2be0404`](https://github.com/langchain-ai/openwork/commit/2be04040719173de8c335509e42fca334ed5f9e4) Fix linter errors in backend & main process - [`ea0c272`](https://github.com/langchain-ai/openwork/commit/ea0c272e81422fe8164653180de80438d439c42d) Fix linter errors in shadcn/ui components - [`539a1f3`](https://github.com/langchain-ai/openwork/commit/539a1f32e994f362ec3a3ee337b5dcc9a800d231) Fix linter errors in backend & main process - [`d6023b8`](https://github.com/langchain-ai/openwork/commit/d6023b814422a7d3ab71213abf1bd5cd5ca962e0) Fix linter errors in feature components - [`705dd42`](https://github.com/langchain-ai/openwork/commit/705dd422d93ee13186bdd05af9fdd98a6b5b623d) Extract selectWorkspaceFolder to separate utility file - [`b0ae813`](https://github.com/langchain-ai/openwork/commit/b0ae8139369f3ab06e56b495a5f870f003cdfba5) Fix ModelSwitcher to derive selectedProviderId instead of setState in effect - [`fc9c803`](https://github.com/langchain-ai/openwork/commit/fc9c8034148dbf89e05b5f494e5cc32fafadaf81) Revert "Fix ModelSwitcher to derive selectedProviderId instead of setState in effect" - [`45f6b8b`](https://github.com/langchain-ai/openwork/commit/45f6b8b38490bffc1528e3fb150fe5eb088ea165) Add TODO comments for eslint-disable directives - [`6a2ad9d`](https://github.com/langchain-ai/openwork/commit/6a2ad9dd53b92f8231b25dccc7f7809480a1b945) Fix linter errors in core libraries and App - [`ae51aa2`](https://github.com/langchain-ai/openwork/commit/ae51aa286ad53d0d23248263dd31e3912a5f0d75) Refactor thread-context into multiple files ### 📊 Changes **50 files changed** (+1111 additions, -1008 deletions) <details> <summary>View changed files</summary> 📝 `bin/cli.js` (+5 -1) 📝 `electron.vite.config.ts` (+2 -2) 📝 `eslint.config.mjs` (+23 -17) 📝 `src/main/agent/runtime.ts` (+3 -1) 📝 `src/main/checkpointer/sqljs-saver.ts` (+9 -1) 📝 `src/main/ipc/agent.ts` (+1 -4) 📝 `src/main/ipc/threads.ts` (+5 -5) 📝 `src/main/services/title-generator.ts` (+8 -8) 📝 `src/main/storage.ts` (+1 -1) 📝 `src/preload/index.d.ts` (+8 -2) 📝 `src/preload/index.ts` (+11 -3) 📝 `src/renderer/src/App.tsx` (+52 -52) 📝 `src/renderer/src/components/chat/ApiKeyDialog.tsx` (+13 -18) 📝 `src/renderer/src/components/chat/ChatContainer.tsx` (+16 -9) 📝 `src/renderer/src/components/chat/ChatTodos.tsx` (+2 -2) 📝 `src/renderer/src/components/chat/ContextUsageIndicator.tsx` (+25 -24) 📝 `src/renderer/src/components/chat/MessageBubble.tsx` (+32 -38) 📝 `src/renderer/src/components/chat/ModelSwitcher.tsx` (+34 -35) 📝 `src/renderer/src/components/chat/StreamingMarkdown.tsx` (+1 -3) 📝 `src/renderer/src/components/chat/ToolCallRenderer.tsx` (+115 -66) _...and 30 more files_ </details> ### 📄 Description Part 4/4 of linter cleanup. Fixes all lint errors in core libraries and App component. **Stack:** #18 → #19 → #20 → **#21** ## Changes - Refactor thread-context into multiple files to satisfy react-refresh rules - Add missing return type annotations - Fix prettier formatting issues ✅ After this PR merges, the codebase will have zero linter errors. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 09:17:51 -05:00
yindo closed this issue 2026-02-16 09:17:52 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
enhancement
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/openwork#34
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?