Release workflow fix #786
Release workflow fix #786
Conversation
amazon1148
requested review from
stea9499,
ColemanRoo,
mrubens and
cte
as code owners
|
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
| @@ -0,0 +1 @@ | |||
| {"root":["./src/app.tsx","./src/index.tsx","./src/setuptests.ts","./src/vite-env.d.ts","./src/__mocks__/vscrui.ts","./src/__mocks__/@vscode/webview-ui-toolkit/react.ts","./src/components/chat/announcement.tsx","./src/components/chat/autoapprovemenu.tsx","./src/components/chat/browsersessionrow.tsx","./src/components/chat/chatrow.tsx","./src/components/chat/chattextarea.tsx","./src/components/chat/chatview.tsx","./src/components/chat/contextmenu.tsx","./src/components/chat/reasoningblock.tsx","./src/components/chat/taskheader.tsx","./src/components/chat/__tests__/chattextarea.test.tsx","./src/components/chat/__tests__/chatview.auto-approve.test.tsx","./src/components/chat/__tests__/chatview.test.tsx","./src/components/common/careticon.tsx","./src/components/common/codeaccordian.tsx","./src/components/common/codeblock.tsx","./src/components/common/demo.tsx","./src/components/common/markdownblock.tsx","./src/components/common/thumbnails.tsx","./src/components/common/vscodebuttonlink.tsx","./src/components/common/__mocks__/codeblock.tsx","./src/components/common/__mocks__/markdownblock.tsx","./src/components/history/historypreview.tsx","./src/components/history/historyview.tsx","./src/components/history/__tests__/historyview.test.tsx","./src/components/mcp/mcpenabledtoggle.tsx","./src/components/mcp/mcpresourcerow.tsx","./src/components/mcp/mcptoolrow.tsx","./src/components/mcp/mcpview.tsx","./src/components/mcp/__tests__/mcptoolrow.test.tsx","./src/components/prompts/promptsview.tsx","./src/components/prompts/__tests__/promptsview.test.tsx","./src/components/settings/apiconfigmanager.tsx","./src/components/settings/apioptions.tsx","./src/components/settings/experimentalfeature.tsx","./src/components/settings/glamamodelpicker.tsx","./src/components/settings/openaimodelpicker.tsx","./src/components/settings/openroutermodelpicker.tsx","./src/components/settings/settingsview.tsx","./src/components/settings/tabnavbar.tsx","./src/components/settings/__tests__/apiconfigmanager.test.tsx","./src/components/settings/__tests__/settingsview.test.tsx","./src/components/ui/button.tsx","./src/components/ui/dropdown-menu.tsx","./src/components/ui/index.ts","./src/components/welcome/welcomeview.tsx","./src/context/extensionstatecontext.tsx","./src/context/__tests__/extensionstatecontext.test.tsx","./src/lib/utils.ts","./src/services/gitservice.ts","./src/stories/button.stories.ts","./src/stories/dropdownmenu.stories.tsx","./src/stories/vscrui/dropdown.stories.tsx","./src/utils/command-validation.ts","./src/utils/context-mentions.ts","./src/utils/format.ts","./src/utils/getlanguagefrompath.ts","./src/utils/highlight.ts","./src/utils/mcp.ts","./src/utils/textmatetohljs.ts","./src/utils/validate.ts","./src/utils/vscode.ts","./src/utils/__tests__/command-validation.test.ts","./src/utils/__tests__/context-mentions.test.ts","../src/shared/extensionmessage.ts","../src/shared/historyitem.ts","../src/shared/webviewmessage.ts","../src/shared/api.ts","../src/shared/array.ts","../src/shared/checkexistapiconfig.ts","../src/shared/combineapirequests.ts","../src/shared/combinecommandsequences.ts","../src/shared/context-mentions.ts","../src/shared/experiments.ts","../src/shared/getapimetrics.ts","../src/shared/mcp.ts","../src/shared/modes.ts","../src/shared/support-prompt.ts","../src/shared/tool-groups.ts","../src/shared/vscodeselectorutils.ts","../src/shared/__tests__/checkexistapiconfig.test.ts","../src/shared/__tests__/modes.test.ts","../src/shared/__tests__/support-prompts.test.ts","../src/shared/__tests__/vscodeselectorutils.test.ts"],"errors":true,"version":"5.7.2"} | |||
There was a problem hiding this comment.
The tsbuildinfo file is an auto-generated cache and should typically be added to .gitignore. Committing it can lead to unnecessary conflicts. Please remove it from version control per our Development Standards: https://www.notion.so/Development-Standards-59febcf8ead647fd9c2ec3f60c22f3df?pvs=4#11869ad2d58180f7b224d6db54dab6b5
| @@ -0,0 +1,4 @@ | |||
| declare module "doctrine" { | |||
| const doctrine: any | |||
There was a problem hiding this comment.
Using 'any' for the 'doctrine' module reduces type safety. If possible, consider providing more specific type definitions or using an existing type declaration package.
| @@ -0,0 +1,9 @@ | |||
| declare module "react" { | |||
There was a problem hiding this comment.
Declaring custom modules for 'react' and 'react/jsx-runtime' with an 'any' type can undermine TypeScript’s benefits and conflict with official type definitions. Prefer using the official '@types/react' package unless there’s a compelling reason otherwise.
Description
Type of change
How Has This Been Tested?
Checklist:
Additional context
Related Issues
Reviewers
Important
Adds a GitHub Actions workflow for VS Code extension release and updates
package.jsonscripts and type declarations.release.ymlto automate VS Code extension release onmainbranch push or manual trigger.package:vscescript inpackage.jsonfor VS Code extension packaging.vscode:prepublishscript to usepackage:vsce.index.d.tsfordoctrineandreactmodules insrc/@types/.This description was created by
for fd7419c. It will automatically update as commits are pushed.