fix: resolve all 6 API param bugs from user report

Author: Revanth Pobala

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "devin-cli"
-version = "1.2.3"
+version = "1.2.4"
 description = "Unofficial CLI for Devin AI - The first AI Software Engineer"
 readme = "README.md"
 authors = [

src/devin_cli/api/v1/knowledge.py

@@ -5,18 +5,19 @@ def list_knowledge():
     return client.get("knowledge")
 
 def create_knowledge(
-    name: str,
-    body: str,
-    trigger_description: str,
+    name: str = None,
+    body: str = None,
+    trigger_description: str = None,
     parent_folder_id: str = None,
     pinned_repo: str = None,
     title: str = None, # v3 fallback
+    trigger: str = None, # v3 fallback
     **kwargs
 ):
     data = {
         "name": name or title,
         "body": body,
-        "trigger_description": trigger_description or ""
+        "trigger_description": trigger_description or trigger or ""
     }
     if parent_folder_id:
         data["parent_folder_id"] = parent_folder_id
@@ -31,15 +32,16 @@ def update_knowledge(
     body: str = None,
     trigger_description: str = None,
     title: str = None, # v3 fallback
+    trigger: str = None, # v3 fallback
     **kwargs
 ):
     data = {}
     if name or title:
         data["name"] = name or title
     if body:
         data["body"] = body
-    if trigger_description:
-        data["trigger_description"] = trigger_description
+    if trigger_description or trigger:
+        data["trigger_description"] = trigger_description or trigger
 
     return client.put(f"knowledge/{knowledge_id}", json=data)
 

src/devin_cli/api/v3/knowledge.py

@@ -5,12 +5,17 @@ def list_knowledge():
     return client.get("knowledge/notes")
 
 def create_knowledge(
-    title: str,
-    body: str,
+    title: str = None,
+    body: str = None,
+    name: str = None,
+    trigger: str = "",
+    trigger_description: str = "",
+    **kwargs
 ):
     data = {
-        "title": title,
+        "name": name or title,
         "body": body,
+        "trigger": trigger or trigger_description or ""
     }
     return client.post("knowledge/notes", json=data)
 
@@ -21,12 +26,18 @@ def update_knowledge(
     note_id: str,
     title: Optional[str] = None,
     body: Optional[str] = None,
+    trigger: Optional[str] = None,
+    name: Optional[str] = None,
+    trigger_description: Optional[str] = None,
+    **kwargs
 ):
     data = {}
-    if title:
-        data["title"] = title
+    if title or name:
+        data["name"] = title or name
     if body:
         data["body"] = body
+    if trigger or trigger_description:
+        data["trigger"] = trigger or trigger_description
 
     return client.put(f"knowledge/notes/{note_id}", json=data)
 

src/devin_cli/api/v3/sessions.py

@@ -27,8 +27,9 @@ def create_session(
 ):
     data = {
         "prompt": prompt,
-        "bypass_approval": bypass_approval,
     }
+    if bypass_approval:
+        data["bypass_approval"] = bypass_approval
     if advanced_mode:
         data["advanced_mode"] = advanced_mode
     if attachment_urls:

src/devin_cli/cli.py

@@ -413,11 +413,12 @@ def list_knowledge_cmd(org: Optional[str] = typer.Option(None, "--org")):
 def create_knowledge_cmd(
     title: str, 
     body: str, 
+    trigger: str = typer.Option("", "--trigger", help="Trigger description"),
     org: Optional[str] = typer.Option(None, "--org"),
 ):
     """Create a knowledge note."""
     if org: config.temporary_org_id = org
-    resp = knowledge.create_knowledge(title, body)
+    resp = knowledge.create_knowledge(title=title, body=body, trigger=trigger)
     console.print(f"[green]Created note:[/green] {resp.get('id')}")
 
 @knowledge_app.command("delete")
@@ -435,13 +436,21 @@ def delete_knowledge_cmd(
 # --- Playbooks ---
 @playbook_app.command("list")
 @handle_api_error
-def list_playbooks_cmd(org: Optional[str] = typer.Option(None, "--org")):
+def list_playbooks_cmd(
+    org: Optional[str] = typer.Option(None, "--org"),
+    json_output: bool = typer.Option(False, "--json", help="Output raw JSON"),
+):
     """List team playbooks."""
     if org: config.temporary_org_id = org
     resp = playbooks.list_playbooks()
     items = resp.get("items", []) if isinstance(resp, dict) else resp
+    
+    if json_output:
+        console.print(json.dumps(items, indent=2))
+        return
+
     table = Table(title="Playbooks")
-    table.add_column("ID", style="cyan")
+    table.add_column("ID", style="cyan", no_wrap=True)
     table.add_column("Title")
     for item in items:
         if isinstance(item, dict):
@@ -950,7 +959,7 @@ def update_knowledge_cmd(
     trigger: Optional[str] = typer.Option(None, "--trigger"),
 ):
     """Update an existing knowledge entry."""
-    knowledge.update_knowledge(knowledge_id, title=name, body=body)
+    knowledge.update_knowledge(knowledge_id, title=name, body=body, trigger=trigger)
     console.print(f"[green]Knowledge {knowledge_id} updated.[/green]")
 
 @app.command("update-playbook")

tests/test_api/test_knowledge.py

@@ -24,19 +24,21 @@ def test_create_knowledge():
     )
     knowledge.create_knowledge(
         title="test",
-        body="content"
+        body="content",
+        trigger="my_trigger"
     )
     assert route.called
     import json
     body = json.loads(route.calls.last.request.read())
-    assert body["title"] == "test"
+    assert body["name"] == "test"
+    assert body["trigger"] == "my_trigger"
 
 @respx.mock
 def test_update_knowledge():
     route = respx.put("https://api.devin.ai/v3/organizations/test_org/knowledge/notes/k_123").mock(
         return_value=Response(200)
     )
-    knowledge.update_knowledge("k_123", title="new")
+    knowledge.update_knowledge("k_123", title="new", trigger="new_trigger")
     assert route.called
 
 @respx.mock