Show DVC Cli Details in DVC Setup

extension/src/setup/index.ts

@@ -8,7 +8,11 @@ import {
 } from 'vscode'
 import { Disposable, Disposer } from '@hediet/std/disposable'
 import isEmpty from 'lodash.isempty'
-import { SetupSection, SetupData as TSetupData } from './webview/contract'
+import {
+  DvcCliDetails,
+  SetupSection,
+  SetupData as TSetupData
+} from './webview/contract'
 import { collectSectionCollapsed } from './collect'
 import { WebviewMessages } from './webview/messages'
 import { validateTokenInput } from './inputBox'
@@ -52,6 +56,7 @@ import { GLOBAL_WEBVIEW_DVCROOT } from '../webview/factory'
 import { ConfigKey, getConfigValue } from '../vscode/config'
 import { getValidInput } from '../vscode/inputBox'
 import { Title } from '../vscode/title'
+import { getOptions } from '../cli/dvc/options'
 
 export type SetupWebviewWebview = BaseWebview<TSetupData>
 
@@ -334,6 +339,21 @@ export class Setup
     return this.sendDataToWebview()
   }
 
+  public async getEnvDetails(): Promise<DvcCliDetails> {
+    const dvcPath = this.config.getCliPath()
+    const pythonBinPath = this.config.getPythonBinPath()
+    const cwd = getFirstWorkspaceFolder()
+
+    const { args, executable } = getOptions(pythonBinPath, dvcPath, cwd || '')
+    const commandArgs = args.join(' ').length === 0 ? '' : ` ${args.join(' ')}`
+    const exampleCommand = executable + commandArgs
+
+    return {
+      exampleCommand,
+      version: cwd ? await this.getCliVersion(cwd) : undefined
+    }
+  }
+
   private async sendDataToWebview() {
     const projectInitialized = this.hasRoots()
     const hasData = this.getHasData()
@@ -348,9 +368,12 @@ export class Setup
 
     const pythonBinPath = await findPythonBinForInstall()
 
+    const dvcCliDetails = await this.getEnvDetails()
+
     this.webviewMessages.sendWebviewMessage({
       canGitInitialize,
       cliCompatible: this.cliCompatible,
+      dvcCliDetails,
       hasData,
       isPythonExtensionInstalled: isPythonExtensionInstalled(),
       isStudioConnected: this.studioIsConnected,

extension/src/setup/webview/contract.ts

@@ -1,6 +1,12 @@
+export type DvcCliDetails = {
+  exampleCommand: string
+  version: string | undefined
+}
+
 export type SetupData = {
   canGitInitialize: boolean
   cliCompatible: boolean | undefined
+  dvcCliDetails: DvcCliDetails
   hasData: boolean | undefined
   isPythonExtensionInstalled: boolean
   isStudioConnected: boolean

extension/src/setup/webview/messages.ts

@@ -35,6 +35,7 @@ export class WebviewMessages {
   public sendWebviewMessage({
     canGitInitialize,
     cliCompatible,
+    dvcCliDetails,
     hasData,
     isPythonExtensionInstalled,
     isStudioConnected,
@@ -48,6 +49,7 @@ export class WebviewMessages {
     void this.getWebview()?.show({
       canGitInitialize,
       cliCompatible,
+      dvcCliDetails,
       hasData,
       isPythonExtensionInstalled,
       isStudioConnected,

extension/src/test/suite/setup/index.test.ts

@@ -218,6 +218,7 @@ suite('Setup Test Suite', () => {
       setup.setCliCompatible(undefined)
       setup.setAvailable(false)
       await setup.setRoots()
+      stub(setup, 'getCliVersion').resolves(undefined)
 
       messageSpy.restore()
       const mockSendMessage = stub(BaseWebview.prototype, 'show')
@@ -238,6 +239,7 @@ suite('Setup Test Suite', () => {
       expect(mockSendMessage).to.be.calledWithExactly({
         canGitInitialize: true,
         cliCompatible: undefined,
+        dvcCliDetails: { exampleCommand: 'dvc', version: undefined },
         hasData: false,
         isPythonExtensionInstalled: false,
         isStudioConnected: false,
@@ -278,6 +280,7 @@ suite('Setup Test Suite', () => {
       expect(mockSendMessage).to.be.calledWithExactly({
         canGitInitialize: true,
         cliCompatible: true,
+        dvcCliDetails: { exampleCommand: 'dvc', version: MIN_CLI_VERSION },
         hasData: false,
         isPythonExtensionInstalled: false,
         isStudioConnected: false,
@@ -324,6 +327,10 @@ suite('Setup Test Suite', () => {
       expect(mockSendMessage).to.be.calledWithExactly({
         canGitInitialize: false,
         cliCompatible: true,
+        dvcCliDetails: {
+          exampleCommand: 'dvc',
+          version: MIN_CLI_VERSION
+        },
         hasData: false,
         isPythonExtensionInstalled: false,
         isStudioConnected: false,
@@ -370,6 +377,10 @@ suite('Setup Test Suite', () => {
       expect(mockSendMessage).to.be.calledWithExactly({
         canGitInitialize: false,
         cliCompatible: true,
+        dvcCliDetails: {
+          exampleCommand: 'dvc',
+          version: MIN_CLI_VERSION
+        },
         hasData: false,
         isPythonExtensionInstalled: false,
         isStudioConnected: false,
@@ -568,6 +579,7 @@ suite('Setup Test Suite', () => {
       mockRunSetup.restore()
       stub(config, 'isPythonExtensionUsed').returns(false)
       stub(config, 'getPythonBinPath').resolves(join('python'))
+      stub(setup, 'getEnvDetails').resolves(undefined)
 
       mockVersion.resetBehavior()
       mockVersion
@@ -627,6 +639,7 @@ suite('Setup Test Suite', () => {
       mockExecuteCommand.restore()
       mockRunSetup.restore()
       stub(config, 'isPythonExtensionUsed').returns(true)
+      stub(setup, 'getEnvDetails').resolves(undefined)
 
       mockVersion.resetBehavior()
       mockVersion.rejects(new Error('no CLI here'))
@@ -762,6 +775,7 @@ suite('Setup Test Suite', () => {
       const mockUpdate = stub()
 
       stub(workspace, 'getConfiguration').returns({
+        get: stub(),
         update: mockUpdate
       } as unknown as WorkspaceConfiguration)
 

webview/src/experiments/components/table/styles.module.scss

@@ -857,12 +857,8 @@ $badge-size: 0.85rem;
 }
 
 .buttonAsLink {
-  @extend %link;
-  background: none;
-  border: none;
-  padding: 0;
+  @extend %buttonAsLink;
   font-size: 0.65rem;
-  cursor: pointer;
 }
 
 .addConfigButton {