fix: make devcontainer ID fields optional in proto schema

- Change id and subagent_id from required to optional bytes
- Remove inline comments from proto file per code review
- Revert TypeScript type assertion workarounds (no longer needed)

The optional keyword makes these fields truly optional in proto3,
which fixes TypeScript type compatibility without requiring unsafe
type assertions.

coderd/database/migrations/000401_add_workspace_agents_index.down.sql

@@ -1 +1 @@
-DROP INDEX IF EXISTS public.workspace_agents_auth_instance_id_deleted_idx;
+DROP INDEX IF EXISTS workspace_agents_auth_instance_id_deleted_idx;

coderd/database/migrations/000401_add_workspace_agents_index.up.sql

@@ -1 +1 @@
-CREATE INDEX IF NOT EXISTS workspace_agents_auth_instance_id_deleted_idx ON public.workspace_agents (auth_instance_id, deleted);
+CREATE INDEX IF NOT EXISTS workspace_agents_auth_instance_id_deleted_idx ON workspace_agents (auth_instance_id, deleted);

provisionersdk/proto/provisioner.proto

@@ -244,6 +244,11 @@ message Devcontainer {
   string workspace_folder = 1;
   string config_path = 2;
   string name = 3;
+  optional bytes id = 4;
+  optional bytes subagent_id = 5;
+  repeated App apps = 6;
+  repeated Script scripts = 7;
+  repeated Env envs = 8;
 }
 
 enum AppOpenIn {

site/e2e/provisionerGenerated.ts

@@ -306,6 +306,11 @@ export interface Devcontainer {
   workspaceFolder: string;
   configPath: string;
   name: string;
+  id?: Uint8Array | undefined;
+  subagentId?: Uint8Array | undefined;
+  apps: App[];
+  scripts: Script[];
+  envs: Env[];
 }
 
 /** App represents a dev-accessible application on the workspace. */
@@ -1095,6 +1100,21 @@ export const Devcontainer = {
     if (message.name !== "") {
       writer.uint32(26).string(message.name);
     }
+    if (message.id !== undefined) {
+      writer.uint32(34).bytes(message.id);
+    }
+    if (message.subagentId !== undefined) {
+      writer.uint32(42).bytes(message.subagentId);
+    }
+    for (const v of message.apps) {
+      App.encode(v!, writer.uint32(50).fork()).ldelim();
+    }
+    for (const v of message.scripts) {
+      Script.encode(v!, writer.uint32(58).fork()).ldelim();
+    }
+    for (const v of message.envs) {
+      Env.encode(v!, writer.uint32(66).fork()).ldelim();
+    }
     return writer;
   },
 };