Initial Setup for Automated Deployments (Panel & Webview)

[ReinierCC]

Putting this PR out with current state of feature development, before it becomes much more complex with full functionality.

Next PR will build out backend functionality on Panel to support UI design which is still being created.

Current functionality is NOT available to user.

[Copilot]

Copilot reviewed 6 out of 15 changed files in this pull request and generated 1 comment.

Files not reviewed (9)

• package.json: Language not supported
• webview-ui/src/AutomatedDeployments/AutomatedDeployments.module.css: Language not supported
• webview-ui/src/manualTest/main.tsx: Evaluated as low risk
• src/commands/utils/acrs.ts: Evaluated as low risk
• src/tree/aksClusterTreeItem.ts: Evaluated as low risk
• webview-ui/src/main.tsx: Evaluated as low risk
• src/webview-contract/webviewTypes.ts: Evaluated as low risk
• src/extension.ts: Evaluated as low risk
• webview-ui/src/AutomatedDeployments/AutomatedDeployments.tsx: Evaluated as low risk

Comments suppressed due to low confidence (4)

webview-ui/src/AutomatedDeployments/state.ts:102

• The getGitHubReposRequest event handler does not update the state in any meaningful way. Consider setting a loading state or handling errors.

getGitHubReposRequest: (state) => ({ ...state, }),

webview-ui/src/AutomatedDeployments/state.ts:109

• The setSelectedGitHubRepo event handler sets the state directly without any validation. Consider adding validation checks before updating the state.

setSelectedGitHubRepo: (state, repo) => ({ ...state, selectedGitHubRepo: repo, }),

webview-ui/src/AutomatedDeployments/state.ts:125

• The setSelectedNamespace event handler sets the state directly without any validation. Consider adding validation checks before updating the state.

setSelectedNamespace: (state, namespace) => ({ ...state, selectedDeploymentNamespace: namespace, }),

src/webview-contract/webviewDefinitions/automatedDeployments.ts:7

• [nitpick] Consider adding a type definition for the elements of the repos array to ensure consistency and avoid potential issues.

repos: string[];

[Tatsinnit]

❤️ Looks good to me!! We have removed a key issue which was previously causing the dependency alert <3 Thanks heaps for this!!

[hsubramanianaks]

looks fine for boilerplate code, but left some minor comments. Hope this get's cleaned up in the upcoming PR's. Thank you Reinier.