feat: make VCN id optional; QoL update (oracle#20)

gebhardtr · gebhardtr · commit 6c0a3d03de3d · 2025-10-09T14:16:12.000-04:00
Signed-off-by: Richard Gebhardt &lt;richard.gebhardt@oracle.com&gt;
diff --git a/oci_networking.py b/oci_networking.py
@@ -70,12 +70,13 @@ def create_subnet(vcn_id: str, compartment_id: str, cidr_block: str, display_nam
 
 
 @mcp.tool
-def list_subnets(compartment_id: str, vcn_id: str):
+def list_subnets(compartment_id: str, vcn_id: str = None):
     networking = get_networking_client()
-    subnets = networking.list_subnets(compartment_id, vcn_id).data
+    subnets = networking.list_subnets(compartment_id, vcn_id=vcn_id).data
     return [
         {
             "subnet_id": subnet.id,
+            "vcn_id": subnet.vcn_id,
             "display_name": subnet.display_name,
             "lifecycle_state": subnet.lifecycle_state,
         }

Original file line number	Diff line number	Diff line change
`@@ -70,12 +70,13 @@ def create_subnet(vcn_id: str, compartment_id: str, cidr_block: str, display_nam`
`70`	`70`
`71`	`71`
`72`	`72`	`@mcp.tool`
`73`		`-def list_subnets(compartment_id: str, vcn_id: str):`
	`73`	`+def list_subnets(compartment_id: str, vcn_id: str = None):`
`74`	`74`	`networking = get_networking_client()`
`75`		`- subnets = networking.list_subnets(compartment_id, vcn_id).data`
	`75`	`+ subnets = networking.list_subnets(compartment_id, vcn_id=vcn_id).data`
`76`	`76`	`return [`
`77`	`77`	`{`
`78`	`78`	`"subnet_id": subnet.id,`
	`79`	`+ "vcn_id": subnet.vcn_id,`
`79`	`80`	`"display_name": subnet.display_name,`
`80`	`81`	`"lifecycle_state": subnet.lifecycle_state,`
`81`	`82`	`}`