fix(cli): read share fields from the v2 share object

The file-share columns pointed at a `sharing` wrapper that v2 does not return.
The share travels under `share` on file metadata (null when unshared) and as the
unwrapped body on the share endpoints, and its flag is `isActive`, not
`enabled`. Every one of those columns was therefore rendering an em-dash on
`files describe`, `files share get`, and `files share set`.

A missing field path renders blank instead of failing, so nothing caught this.
Added a rendering test over both surfaces; it fails if a path stops resolving.
`hasPassword` is surfaced on the two share commands while they are being fixed.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JNFjchn6dcM7xevh34PKHE

Author: TheodoreSpeaks

packages/sim-cli/src/contract/commands.ts

@@ -469,10 +469,12 @@ export const CLI_CONTRACT: CliContract = {
       { header: 'uploaded by', path: 'uploadedByEmail' },
       { header: 'uploaded', path: 'uploadedAt', format: 'timestamp' },
       { header: 'updated', path: 'updatedAt', format: 'timestamp' },
-      { header: 'shared', path: 'sharing.enabled', format: 'bool' },
-      { header: 'share URL', path: 'sharing.url' },
-      { header: 'share auth', path: 'sharing.authType' },
-      { header: 'allowed emails', path: 'sharing.allowedEmails', format: 'count' },
+      // v2 returns the share under `share` (null when unshared), and its flag
+      // is `isActive`.
+      { header: 'shared', path: 'share.isActive', format: 'bool' },
+      { header: 'share URL', path: 'share.url' },
+      { header: 'share auth', path: 'share.authType' },
+      { header: 'allowed emails', path: 'share.allowedEmails', format: 'count' },
     ],
   },
   moveFileItems: {
@@ -500,14 +502,17 @@ export const CLI_CONTRACT: CliContract = {
       encoding: { choices: ['utf-8', 'base64'], describe: 'Content encoding' },
     },
   },
+  // Both share commands return the share itself as `data`, which the runtime
+  // unwraps, so these fields sit at the top level rather than under a wrapper.
   getFileShare: {
     command: 'files share get',
     describe: 'Show a file’s share settings',
     fields: [
-      { header: 'shared', path: 'sharing.enabled', format: 'bool' },
-      { header: 'URL', path: 'sharing.url' },
-      { header: 'auth', path: 'sharing.authType' },
-      { header: 'allowed emails', path: 'sharing.allowedEmails', format: 'count' },
+      { header: 'shared', path: 'isActive', format: 'bool' },
+      { header: 'URL', path: 'url' },
+      { header: 'auth', path: 'authType' },
+      { header: 'password set', path: 'hasPassword', format: 'bool' },
+      { header: 'allowed emails', path: 'allowedEmails', format: 'count' },
     ],
   },
   // v2 folds share and unshare into one PATCH; `--is-active false` disables it,
@@ -519,10 +524,11 @@ export const CLI_CONTRACT: CliContract = {
       allowedEmails: { list: true },
     },
     fields: [
-      { header: 'shared', path: 'sharing.enabled', format: 'bool' },
-      { header: 'URL', path: 'sharing.url' },
-      { header: 'auth', path: 'sharing.authType' },
-      { header: 'allowed emails', path: 'sharing.allowedEmails', format: 'count' },
+      { header: 'shared', path: 'isActive', format: 'bool' },
+      { header: 'URL', path: 'url' },
+      { header: 'auth', path: 'authType' },
+      { header: 'password set', path: 'hasPassword', format: 'bool' },
+      { header: 'allowed emails', path: 'allowedEmails', format: 'count' },
     ],
   },
 

packages/sim-cli/src/runtime/build.test.ts

@@ -945,6 +945,44 @@ describe('contract-selected list rendering', () => {
     expect(secrets[0]).toContain('STRIPE_API_KEY')
     expect(secrets[0]).toContain('workspace')
   })
+
+  /**
+   * A field path that misses renders as an em-dash rather than failing, so a
+   * renamed response key is invisible until someone reads the output. v2 nests
+   * the share under `share` and calls the flag `isActive`; the CLI briefly read
+   * a `sharing` wrapper and silently showed nothing for all four columns.
+   */
+  it('reads share fields from the v2 share object, not a sharing wrapper', async () => {
+    const described = (
+      await lines(['files', 'describe', 'file_1'], {
+        id: 'file_1',
+        name: 'notes.txt',
+        uploadedByEmail: 'ada@example.com',
+        share: {
+          isActive: true,
+          url: 'https://sim.ai/s/tok_1',
+          authType: 'email',
+          hasPassword: false,
+          allowedEmails: ['ada@example.com'],
+        },
+      })
+    ).join('\n')
+    expect(described).toContain('https://sim.ai/s/tok_1')
+    expect(described).toContain('email')
+    expect(described).toContain('ada@example.com')
+
+    const share = (
+      await lines(['files', 'share', 'get', 'file_1'], {
+        isActive: true,
+        url: 'https://sim.ai/s/tok_2',
+        authType: 'sso',
+        hasPassword: true,
+        allowedEmails: ['ada@example.com', 'grace@example.com'],
+      })
+    ).join('\n')
+    expect(share).toContain('https://sim.ai/s/tok_2')
+    expect(share).toContain('sso')
+  })
 })
 
 describe('pagination slot', () => {