codeberg-cli 0.4.2__tar.gz → 0.5.0__tar.gz

codeberg_cli-0.5.0/.claude/workflows/forgejo-surface.mjs

@@ -0,0 +1,488 @@
+export const meta = {
+  name: 'forgejo-surface',
+  description: 'Implement the remaining Forgejo API surface in cb (7 groups), each adversarially reviewed and tested',
+  phases: [
+    { title: 'Implement', detail: 'one agent per surface group writes routes + tests' },
+    { title: 'Review', detail: 'code-reviewer audits each group against spec + conventions' },
+    { title: 'Fix', detail: 'apply confirmed defects per group' },
+  ],
+}
+
+// ---- Shared brief: conventions every implementer + reviewer must honor ----
+const CONVENTIONS = `
+PROJECT: codeberg-cli ("cb"), a Python CLI built on the xclif framework. Repo root:
+/Users/bryanhu/Developer/current/XClif/codeberg-cli
+Source lives under src/codeberg_cli/routes/. Tests under tests/.
+
+VERIFIED ROUTING RULES (do not deviate — confirmed from xclif source):
+- One file per command. The function is ALWAYS named \`_\` and decorated
+  @command("name") or @command("name", "alias").
+- A subcommand GROUP is a directory containing __init__.py with @command("group")
+  and a one-line docstring (no body). Example routes/repo/branch/__init__.py:
+      from xclif import command
+      @command("branch")
+      def _() -> None:
+          """Manage branches — list, create, view, and delete."""
+- CRITICAL: the DIRECTORY NAME MUST EXACTLY EQUAL the group's @command token.
+  xclif derives intermediate namespace tokens from the directory name but mounts
+  the __init__ command by its decorator name; if they differ the tree breaks.
+  => Directory names must be valid Python identifiers AND the CLI token. NO HYPHENS
+     in directory names. (e.g. use dir "mirror/" with @command("mirror"), never
+     "push-mirror".)
+- xclif autodiscovers the route tree by walking the package. NO central registration.
+- Args:    name: Annotated[int, Arg(description="...")]
+- Options: name: Annotated[str, Option(description="...", name="cli-name")] = default
+  (option CLI names MAY contain hyphens; only DIRECTORY names may not.)
+- Imports come from: from xclif import Arg, Option, command  (and Command in tests).
+
+HELPERS (from codeberg_cli.helpers):
+- client = require_client()                  # authenticated httpx wrapper or exits
+- is_json_mode()                             # True if --json passed
+- output(obj)                                # prints obj as JSON when in json mode
+- print_table(cols, rows, json_data=...)     # table, or JSON when --json
+- get_web_base_url()                         # web UI base url for links
+Client verbs: client.get(path, params=...), client.post(path, data=...),
+client.patch(path, data=...), client.put(path), client.delete(path).
+DELETE returns None. GET/POST/PATCH/PUT return parsed JSON (dict or list).
+
+REPO INFERENCE (copy VERBATIM into every repo-scoped command; skip for user-scoped):
+    from codeberg_cli.git import infer_repo
+    ...
+    if not repo:
+        inferred = infer_repo()
+        if not inferred:
+            rich.print("[bold red]Error:[/bold red] No repo specified and not in a git directory")
+            return 1
+        repo = inferred
+The repo option is always:
+    repo: Annotated[str, Option(description="Repository (owner/repo)", name="repo")] = ""
+
+OUTPUT / ERROR CONVENTIONS:
+- List commands: build rows + json_rows, then print_table([...], rows, json_data=json_rows).
+  Handle empty: rich.print("[dim]No ...[/dim]"); return
+- Detail commands: if is_json_mode(): output(obj); return  — then rich.print human view.
+- Success: rich.print("[bold green]<Verb>[/bold green] ...")
+- Errors: rich.print("[bold red]Error:[/bold red] ...") then return 1
+- import rich at top for rich.print.
+
+MULTIPART UPLOAD (only for attachments) — mirror routes/release/upload.py EXACTLY:
+    token = client._client.headers.get("Authorization", "").replace("Bearer ", "")
+    headers = {"Authorization": f"Bearer {token}"}
+    response = httpx.post(f"{client.base_url}/<path>", headers=headers,
+                          files={"attachment": (file_path.name, content)})
+
+TESTS (tests/test_<area>.py) — must NOT hit the network. Pattern:
+    from xclif import Command
+    def test_<area>_commands_are_commands():
+        from codeberg_cli.routes.<...>.create import _ as create_cmd
+        ...
+        for cmd in [create_cmd, ...]:
+            assert isinstance(cmd, Command)
+    def test_<area>_create_name():
+        from codeberg_cli.routes.<...>.create import _ as cmd
+        assert cmd.name == "create"
+  Add an alias assertion where an alias exists (e.g. assert "ls" in cmd.aliases).
+
+REFERENCE FILES TO MIRROR (read them before writing):
+- src/codeberg_cli/routes/issue/list.py        (list/table + repo inference + --json)
+- src/codeberg_cli/routes/issue/subscribe.py   (arg + put + simple success)
+- src/codeberg_cli/routes/pr/view.py           (detail view + is_json_mode + web flag)
+- src/codeberg_cli/routes/release/upload.py    (multipart upload)
+- src/codeberg_cli/routes/repo/branch/__init__.py (group __init__.py shape)
+- tests/test_issue.py                          (test shape)
+
+Endpoint spec ground truth: /tmp/forgejo_swagger.json (Forgejo OpenAPI). When unsure
+of a request/response body shape, read the relevant path's schema from that file with
+python -c / a small script — do NOT guess field names.
+`
+
+// ---- Per-group implementation briefs ----
+const GROUPS = [
+  {
+    key: 'keys',
+    label: 'keys (ssh/gpg)',
+    files: [
+      'src/codeberg_cli/routes/keys/__init__.py',
+      'src/codeberg_cli/routes/keys/ssh/__init__.py',
+      'src/codeberg_cli/routes/keys/ssh/list.py',
+      'src/codeberg_cli/routes/keys/ssh/add.py',
+      'src/codeberg_cli/routes/keys/ssh/delete.py',
+      'src/codeberg_cli/routes/keys/gpg/__init__.py',
+      'src/codeberg_cli/routes/keys/gpg/list.py',
+      'src/codeberg_cli/routes/keys/gpg/add.py',
+      'src/codeberg_cli/routes/keys/gpg/delete.py',
+      'tests/test_keys.py',
+    ],
+    brief: `
+Implement \`cb keys\` — user SSH and GPG key management. USER-SCOPED: NO repo inference.
+
+Group __init__.py files:
+- routes/keys/__init__.py        @command("keys")  docstring "Manage your SSH and GPG keys."
+- routes/keys/ssh/__init__.py    @command("ssh")   docstring "Manage SSH keys."
+- routes/keys/gpg/__init__.py    @command("gpg")   docstring "Manage GPG keys."
+
+cb keys ssh list  (alias "ls"): GET /user/keys → table [ID, Title, Fingerprint] (+json_data
+  with id,title,fingerprint,created_at). Empty → "[dim]No SSH keys.[/dim]".
+cb keys ssh add: args  title (Arg), key (Arg, the public key string). Also accept
+  --key-file option to read the key from a file path (if given, read file, strip).
+  POST /user/keys data {"title": title, "key": key}. Success: green "Added SSH key" + id.
+cb keys ssh delete: arg id (int). DELETE /user/keys/{id}. Success "Deleted SSH key #<id>".
+
+cb keys gpg list (alias "ls"): GET /user/gpg_keys → table [ID, Key ID, Can Sign] (+json_data
+  id, key_id, can_sign, created_at). Empty → "[dim]No GPG keys.[/dim]".
+cb keys gpg add: arg armored_key (Arg) OR --key-file option (read file). POST /user/gpg_keys
+  data {"armored_public_key": <value>}. Success green "Added GPG key" + id.
+cb keys gpg delete: arg id (int). DELETE /user/gpg_keys/{id}. Success "Deleted GPG key #<id>".
+
+tests/test_keys.py: assert each leaf is a Command; assert names; assert "ls" alias on both list cmds.
+`,
+  },
+  {
+    key: 'pr',
+    label: 'pr reviews + status',
+    files: [
+      'src/codeberg_cli/routes/pr/review.py',
+      'src/codeberg_cli/routes/pr/reviews.py',
+      'src/codeberg_cli/routes/pr/status.py',
+      'tests/test_pr.py (APPEND new tests; do not delete existing)',
+    ],
+    brief: `
+Extend the existing \`cb pr\` group (routes/pr/) with three new leaf commands. Do NOT
+create a subdirectory — these are siblings of routes/pr/view.py etc. Read routes/pr/view.py
+first for the exact repo-inference + json + arg pattern.
+
+cb pr review <id>: arg id (int, "PR number"). Options:
+  --approve (bool), --request-changes (bool), --comment (bool), --body (str, default "").
+  Map to event: approve→"APPROVED", request-changes→"REQUEST_CHANGES", else "COMMENT".
+  If more than one of the three flags is set → error "[bold red]Error:[/bold red] Choose at
+  most one of --approve/--request-changes/--comment" return 1.
+  POST /repos/{repo}/pulls/{id}/reviews  data {"event": <event>, "body": body}.
+  (For REQUEST_CHANGES/COMMENT a body is recommended; do not hard-require it.)
+  Success: green "Submitted <event> review on PR #<id>".
+
+cb pr reviews <id> (alias none): arg id (int). GET /repos/{repo}/pulls/{id}/reviews →
+  table [ID, Reviewer, State, Submitted] from each review's id, user.login, state,
+  submitted_at. json_data list of those fields. Empty → "[dim]No reviews on PR #<id>.[/dim]".
+
+cb pr status <id>: arg id (int). First GET /repos/{repo}/pulls/{id} to get head sha
+  (pr["head"]["sha"]). Then GET /repos/{repo}/commits/{sha}/status → combined status object
+  with keys "state" and "statuses" (list of {context, state, target_url, description}).
+  if is_json_mode(): output(that object); return.
+  Human: rich.print combined state colored (success→green, pending→yellow, error/failure→red),
+  then one line per status context: "<state> <context>". If no statuses: "[dim]No CI status[/dim]".
+
+APPEND to tests/test_pr.py (keep existing tests intact): assert review/reviews/status are
+Commands and have the right .name.
+
+Verify exact field names against /tmp/forgejo_swagger.json (PullReview, CombinedStatus, CommitStatus).
+`,
+  },
+  {
+    key: 'hook',
+    label: 'repo hooks',
+    files: [
+      'src/codeberg_cli/routes/repo/hook/__init__.py',
+      'src/codeberg_cli/routes/repo/hook/list.py',
+      'src/codeberg_cli/routes/repo/hook/view.py',
+      'src/codeberg_cli/routes/repo/hook/create.py',
+      'src/codeberg_cli/routes/repo/hook/delete.py',
+      'src/codeberg_cli/routes/repo/hook/test.py',
+      'tests/test_repo_hook.py',
+    ],
+    brief: `
+Implement \`cb repo hook\` — repository webhooks. Directory routes/repo/hook/ (dir name "hook"
+== @command("hook")). All commands are repo-scoped (use the verbatim repo-inference block).
+
+routes/repo/hook/__init__.py: @command("hook") docstring "Manage repository webhooks."
+
+cb repo hook list (alias "ls"): GET /repos/{repo}/hooks → table [ID, Type, URL, Active]
+  (url from h["config"]["url"]). json_data id,type,active,events,config. Empty → "[dim]No webhooks.[/dim]".
+cb repo hook view <id>: arg id (int). GET /repos/{repo}/hooks/{id}; if is_json_mode(): output; return.
+  Human: print id, type, active, events (comma-joined), config url + content_type.
+cb repo hook create: options --type (str, default "forgejo"), --url (str, REQUIRED — if empty
+  error+return 1), --events (str, comma-separated, default "push"), --content-type (str,
+  name="content-type", default "json"), --secret (str default ""), --active (bool default True).
+  POST /repos/{repo}/hooks data {"type": type, "active": active,
+     "events": [e.strip() for e in events.split(",") if e.strip()],
+     "config": {"url": url, "content_type": content_type, **({"secret": secret} if secret else {})}}.
+  Success green "Created webhook #<id>".
+cb repo hook delete: arg id (int). DELETE /repos/{repo}/hooks/{id}. Success "Deleted webhook #<id>".
+cb repo hook test: arg id (int). POST /repos/{repo}/hooks/{id}/tests (data None).
+  Success green "Triggered test delivery for webhook #<id>".
+
+tests/test_repo_hook.py: Commands + names + "ls" alias on list. Verify Hook/CreateHookOption
+shapes against /tmp/forgejo_swagger.json.
+`,
+  },
+  {
+    key: 'wiki',
+    label: 'repo wiki',
+    files: [
+      'src/codeberg_cli/routes/repo/wiki/__init__.py',
+      'src/codeberg_cli/routes/repo/wiki/list.py',
+      'src/codeberg_cli/routes/repo/wiki/view.py',
+      'src/codeberg_cli/routes/repo/wiki/create.py',
+      'src/codeberg_cli/routes/repo/wiki/edit.py',
+      'src/codeberg_cli/routes/repo/wiki/delete.py',
+      'tests/test_repo_wiki.py',
+    ],
+    brief: `
+Implement \`cb repo wiki\` — repository wiki pages. Directory routes/repo/wiki/. Repo-scoped.
+
+routes/repo/wiki/__init__.py: @command("wiki") docstring "Manage wiki pages."
+
+cb repo wiki list (alias "ls"): GET /repos/{repo}/wiki/pages → table [Title, Last Commit]
+  (title, last_commit.sha[:8] if present else ""). json_data title, sub_url, last_commit.
+  Empty → "[dim]No wiki pages.[/dim]".
+cb repo wiki view <page>: arg page (str, "Page name"). GET /repos/{repo}/wiki/page/{page}
+  (URL-encode the page name with urllib.parse.quote, safe=""). Response has content_base64.
+  if is_json_mode(): output(resp); return. Human: print title bold, then decode:
+  import base64; text = base64.b64decode(resp["content_base64"]).decode("utf-8", "replace");
+  rich.print(text).
+cb repo wiki create: options --title (str REQUIRED → error+return 1 if empty),
+  --content (str, default ""), --message (str default "", commit message).
+  POST /repos/{repo}/wiki/new data {"title": title, "content_base64":
+     base64.b64encode(content.encode()).decode(), **({"message": message} if message else {})}.
+  Success green "Created wiki page '<title>'".
+cb repo wiki edit <page>: arg page (str). options --title (str default "" → keep), --content (str
+  default "" — only send if provided), --message. Build data with content_base64 from content,
+  title if given. PATCH /repos/{repo}/wiki/page/{page}. Success "Updated wiki page '<page>'".
+cb repo wiki delete <page>: arg page. DELETE /repos/{repo}/wiki/page/{page}. Success "Deleted wiki page '<page>'".
+
+tests/test_repo_wiki.py: Commands + names + "ls" alias on list. Verify WikiPage /
+CreateWikiPageOptions field names (content_base64, message, title) against /tmp/forgejo_swagger.json.
+`,
+  },
+  {
+    key: 'mirror',
+    label: 'push mirrors',
+    files: [
+      'src/codeberg_cli/routes/repo/mirror/__init__.py',
+      'src/codeberg_cli/routes/repo/mirror/list.py',
+      'src/codeberg_cli/routes/repo/mirror/create.py',
+      'src/codeberg_cli/routes/repo/mirror/delete.py',
+      'src/codeberg_cli/routes/repo/mirror/sync.py',
+      'tests/test_push_mirror.py',
+    ],
+    brief: `
+Implement \`cb repo mirror\` — push mirrors. Directory routes/repo/mirror/ (token "mirror";
+DO NOT use a hyphenated dir name). Repo-scoped.
+
+routes/repo/mirror/__init__.py: @command("mirror") docstring "Manage push mirrors."
+
+cb repo mirror list (alias "ls"): GET /repos/{repo}/push_mirrors → table
+  [Name, Remote, Interval, Last Update] (remote_name, remote_address, interval, last_update).
+  json_data those fields + sync_on_commit. Empty → "[dim]No push mirrors.[/dim]".
+cb repo mirror create: options --remote (str REQUIRED → error+return 1 if empty, the remote
+  repo address/URL), --interval (str default "8h0m0s"), --sync-on-commit (bool name="sync-on-commit"
+  default False), --username (str default ""), --password (str default "").
+  POST /repos/{repo}/push_mirrors data {"remote_address": remote, "interval": interval,
+     "sync_on_commit": sync_on_commit, **({"remote_username": username} if username else {}),
+     **({"remote_password": password} if password else {})}.
+  Success green "Created push mirror to <remote>".
+cb repo mirror delete <name>: arg name (str, the remote_name). DELETE /repos/{repo}/push_mirrors/{name}.
+  Success "Deleted push mirror '<name>'".
+cb repo mirror sync: POST /repos/{repo}/push_mirrors-sync (data None). Success green
+  "Triggered push mirror sync for <repo>".
+
+tests/test_push_mirror.py: Commands + names + "ls" alias on list. Verify
+CreatePushMirrorOption field names against /tmp/forgejo_swagger.json.
+`,
+  },
+  {
+    key: 'notify',
+    label: 'notifications (migrate to group)',
+    files: [
+      'src/codeberg_cli/routes/notifications.py (DELETE this file)',
+      'src/codeberg_cli/routes/notify/__init__.py',
+      'src/codeberg_cli/routes/notify/list.py',
+      'src/codeberg_cli/routes/notify/read.py',
+      'src/codeberg_cli/routes/notify/thread.py',
+      'tests/test_notifications.py (REWRITE for new module paths)',
+    ],
+    brief: `
+Migrate notifications into a \`cb notify\` group. Read the EXISTING file first:
+src/codeberg_cli/routes/notifications.py — it has the current listing logic.
+
+STEPS:
+1. DELETE src/codeberg_cli/routes/notifications.py (it becomes the group's list command).
+2. routes/notify/__init__.py: @command("notify") docstring "View and manage notifications."
+3. routes/notify/list.py: move the EXISTING listing logic here. Decorate
+   @command("list", "notifications") so \`cb notify list\` works and \`cb notify notifications\`
+   is an alias. (Note: this is now nested under \`notify\`; the old top-level \`cb notifications\`
+   becomes \`cb notify notifications\` / \`cb notify list\` — that's the intended migration.)
+   Keep the same options (--limit, --all) and table output exactly.
+4. routes/notify/read.py: @command("read"). Options --all (bool default False),
+   --repo (str default "", the repo-inference block is OPTIONAL here — only infer if --repo
+   given OR a last-read approach). Behavior:
+     - if repo (explicitly given): PUT /repos/{repo}/notifications  (mark that repo's read)
+     - else: PUT /notifications  (mark ALL read)
+   Both accept query param "all=true" semantics via Forgejo's "status-types"; simplest: call
+   client.put with no body. Success green "Marked notifications as read".
+   (Use client.put(path) — it sends no body which is fine.)
+5. routes/notify/thread.py: @command("thread"). arg id (int, "Thread ID"). option --read (bool
+   default False). If --read: PATCH /notifications/threads/{id} → success "Marked thread #<id> read".
+   Else: GET /notifications/threads/{id}; if is_json_mode(): output; return; human-print subject
+   type/title and repository full_name.
+
+REWRITE tests/test_notifications.py: import from codeberg_cli.routes.notify.list / read / thread.
+Assert each is a Command, assert list_cmd.name == "list" and "notifications" in list_cmd.aliases,
+assert read/thread names.
+
+IMPORTANT: after deleting notifications.py, make sure nothing else imports it. grep the repo
+for "routes.notifications" and "import notifications" and fix/none. The old test file referenced
+the old module — that's why you rewrite it.
+`,
+  },
+  {
+    key: 'issue-extras',
+    label: 'issue reactions/deps/attachments',
+    files: [
+      'src/codeberg_cli/routes/issue/react.py',
+      'src/codeberg_cli/routes/issue/reactions.py',
+      'src/codeberg_cli/routes/issue/deps/__init__.py',
+      'src/codeberg_cli/routes/issue/deps/list.py',
+      'src/codeberg_cli/routes/issue/deps/add.py',
+      'src/codeberg_cli/routes/issue/deps/remove.py',
+      'src/codeberg_cli/routes/issue/assets/__init__.py',
+      'src/codeberg_cli/routes/issue/assets/list.py',
+      'src/codeberg_cli/routes/issue/assets/upload.py',
+      'src/codeberg_cli/routes/issue/assets/delete.py',
+      'tests/test_issue.py (APPEND; keep existing tests)',
+    ],
+    brief: `
+Extend \`cb issue\` with reactions, dependencies, attachments. Read routes/issue/list.py,
+routes/issue/subscribe.py, routes/release/upload.py first. All repo-scoped.
+
+REACTIONS (sibling leaf files, NOT a subgroup):
+- routes/issue/react.py @command("react"): args index (int "Issue number"), content (str
+  "Reaction emoji, e.g. +1, -1, heart, laugh"). option --remove (bool default False).
+  if remove: DELETE /repos/{repo}/issues/{index}/reactions with the body — BUT client.delete
+  takes no body. So for remove use client._request? NO — instead call client.post for add and
+  for remove use the framework: the API needs a JSON body {"content": content} on DELETE.
+  Use httpx via the client's underlying request for DELETE-with-body:
+      client._client.request("DELETE", f"/repos/{repo}/issues/{index}/reactions",
+                             json={"content": content})
+  (check resp; on non-2xx rich red error + return 1). For add: client.post(
+  f"/repos/{repo}/issues/{index}/reactions", data={"content": content}).
+  Success: "Added reaction :<content>:" / "Removed reaction :<content>:".
+- routes/issue/reactions.py @command("reactions"): arg index (int). GET
+  /repos/{repo}/issues/{index}/reactions → table [User, Reaction] (user.login, content).
+  json_data list. Empty → "[dim]No reactions.[/dim]".
+
+DEPENDENCIES — group routes/issue/deps/ (dir "deps"):
+- deps/__init__.py @command("deps") docstring "Manage issue dependencies (blocked-by)."
+- deps/list.py @command("list","ls"): arg index (int). GET /repos/{repo}/issues/{index}/dependencies
+  → table [#, Title, State] (number,title,state). json_data. Empty → "[dim]No dependencies.[/dim]".
+- deps/add.py @command("add"): args index (int "Issue number"), depends_on (int "Issue number
+  this one depends on"). POST /repos/{repo}/issues/{index}/dependencies data
+  {"index": depends_on}. (Same-repo dependency.) Success "Issue #<index> now depends on #<depends_on>".
+- deps/remove.py @command("remove"): args index, depends_on. DELETE-with-body like reactions:
+  client._client.request("DELETE", ".../dependencies", json={"index": depends_on}); check resp.
+  Success "Removed dependency #<depends_on> from issue #<index>".
+
+ATTACHMENTS — group routes/issue/assets/ (dir "assets"):
+- assets/__init__.py @command("assets") docstring "Manage issue attachments."
+- assets/list.py @command("list","ls"): arg index (int). GET /repos/{repo}/issues/{index}/assets
+  → table [ID, Name, Size, Download] (id,name,size,browser_download_url). json_data. Empty msg.
+- assets/upload.py @command("upload"): args index (int), file (str). Mirror release/upload.py
+  multipart EXACTLY but POST to f"{client.base_url}/repos/{repo}/issues/{index}/assets" with
+  files={"attachment": (file_path.name, content)}. Check file exists first (red error+return 1).
+  Success green "Uploaded <name> to issue #<index>".
+- assets/delete.py @command("delete"): args index (int), attachment_id (int). DELETE
+  /repos/{repo}/issues/{index}/assets/{attachment_id}. Success "Deleted attachment #<attachment_id>".
+
+APPEND to tests/test_issue.py (DO NOT remove existing tests): import the new commands, assert
+Commands, assert names, assert "ls" alias on deps/list and assets/list.
+
+Verify Reaction / IssueDependency / Attachment shapes against /tmp/forgejo_swagger.json.
+`,
+  },
+]
+
+phase('Implement')
+log(`Implementing ${GROUPS.length} Forgejo surface groups in parallel, each reviewed + fixed independently.`)
+
+const REVIEW_SCHEMA = {
+  type: 'object',
+  additionalProperties: false,
+  properties: {
+    defects: {
+      type: 'array',
+      items: {
+        type: 'object',
+        additionalProperties: false,
+        properties: {
+          file: { type: 'string' },
+          severity: { type: 'string', enum: ['critical', 'major', 'minor'] },
+          issue: { type: 'string' },
+          fix: { type: 'string' },
+        },
+        required: ['file', 'severity', 'issue', 'fix'],
+      },
+    },
+    summary: { type: 'string' },
+  },
+  required: ['defects', 'summary'],
+}
+
+const results = await pipeline(
+  GROUPS,
+  // Stage 1: implement
+  (g) =>
+    agent(
+      `${CONVENTIONS}\n\n=== YOUR TASK: implement group "${g.label}" ===\n${g.brief}\n\n` +
+        `Write every file listed. Create __init__.py for any new directory. Run NO build/test ` +
+        `commands yourself beyond reading files and the swagger spec. When done, return a short ` +
+        `confirmation listing the files you wrote.`,
+      { label: `impl:${g.key}`, phase: 'Implement' }
+    ).then((res) => ({ g, implReport: res })),
+  // Stage 2: adversarial review
+  ({ g }) =>
+    agent(
+      `${CONVENTIONS}\n\n=== ADVERSARIAL REVIEW of group "${g.label}" ===\n` +
+        `These files were just written:\n${g.files.map((f) => '  - ' + f).join('\n')}\n\n` +
+        `Read each one. Check, with skepticism:\n` +
+        `1. ROUTING: every group directory name EXACTLY equals its @command token (no hyphens in dir names). ` +
+        `Leaf functions named \`_\`, decorated correctly.\n` +
+        `2. API CORRECTNESS: paths and request/response field names match the Forgejo spec at ` +
+        `/tmp/forgejo_swagger.json (read the relevant schemas — do not assume). Flag wrong paths, ` +
+        `wrong field names, wrong HTTP verbs, missing required body fields.\n` +
+        `3. CONVENTIONS: repo-inference block present & verbatim for repo-scoped cmds (absent for ` +
+        `user-scoped); require_client() used; --json handled via is_json_mode()/output()/print_table; ` +
+        `errors use the red-error + return 1 pattern; imports correct (Arg/Option/command/rich/httpx).\n` +
+        `4. DELETE-with-body and multipart cases use the documented workaround correctly.\n` +
+        `5. TESTS exist, assert Command-ness + names + aliases, and do NOT hit the network.\n` +
+        `6. For the notify group: notifications.py is deleted and nothing imports it.\n` +
+        `Report ONLY real defects with a concrete fix each. If clean, return empty defects.`,
+      { label: `review:${g.key}`, phase: 'Review', schema: REVIEW_SCHEMA, agentType: 'feature-dev:code-reviewer' }
+    ).then((review) => ({ g, review })),
+  // Stage 3: fix confirmed defects
+  ({ g, review }) => {
+    const real = (review?.defects || []).filter((d) => d.severity !== 'minor')
+    if (!real.length) {
+      return Promise.resolve({ g, review, fixReport: 'no actionable defects' })
+    }
+    return agent(
+      `${CONVENTIONS}\n\n=== APPLY FIXES to group "${g.label}" ===\n` +
+        `A reviewer found these defects. Fix each in the listed files ONLY (do not touch other ` +
+        `groups). After fixing, re-read to confirm. Defects:\n` +
+        real
+          .map((d, i) => `${i + 1}. [${d.severity}] ${d.file}: ${d.issue}\n   FIX: ${d.fix}`)
+          .join('\n') +
+        `\n\nReturn a short confirmation of what you changed.`,
+      { label: `fix:${g.key}`, phase: 'Fix' }
+    ).then((fixReport) => ({ g, review, fixReport }))
+  }
+)
+
+const clean = results.filter(Boolean)
+return {
+  groups: clean.map((r) => ({
+    group: r.g.label,
+    reviewSummary: r.review?.summary,
+    defectCount: (r.review?.defects || []).length,
+    actionableFixed: (r.review?.defects || []).filter((d) => d.severity !== 'minor').length,
+    fix: r.fixReport,
+  })),
+}

{codeberg_cli-0.4.2 → codeberg_cli-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeberg-cli
-Version: 0.4.2
+Version: 0.5.0
 Summary: A Forgejo CLI — works with Codeberg and any Forgejo instance
 Project-URL: Homepage, https://codeberg.org/ThatXliner/codeberg-cli
 Project-URL: Repository, https://codeberg.org/ThatXliner/codeberg-cli
@@ -93,7 +93,7 @@ cb config set base_url "https://git.example.com/api/v1"  # Self-hosted Forgejo
 
 ## Comparison
 
-Here's how `cb` stacks up against other Forgejo CLI tools. *Last updated: May 2026.*
+Here's how `cb` stacks up against other Forgejo CLI tools. *Last updated: June 2026.*
 
 TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 
@@ -116,12 +116,12 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Delete | ✅ | — | — | — | — |
 | Pin/Unpin | ✅ | — | — | ✅ | — |
 | Search | ✅ | ✅ | — | ✅ | — |
-| Attachments | — | — | — | ✅ | — |
+| Attachments | ✅ | — | — | ✅ | — |
 | Labels (manage on issue) | ✅ | — | — | ✅ | ✅ |
-| Reactions | — | — | — | — | — |
+| Reactions | ✅ | — | — | — | — |
 | Subscriptions | ✅ | — | — | ✅ | — |
 | Tracked times | ✅ | — | — | ✅ | ✅ |
-| Dependencies | — | — | — | — | — |
+| Dependencies | ✅ | — | — | — | — |
 | Deadline | — | — | — | ✅ | — |
 | Templates | — | ✅ | — | — | — |
 
@@ -139,11 +139,11 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Checkout | ✅ | ✅ | ✅ | ✅ | ✅ |
 | Commits | ✅ | ✅ | — | — | ✅ |
 | Files/changed | ✅ | ✅ | — | — | — |
-| Reviews | — | ✅ | — | ✅ | ✅ |
+| Reviews | ✅ | ✅ | — | ✅ | ✅ |
 | Diff/Patch | ✅ | ✅ | — | — | ✅ |
 | Update branch | ✅ | — | — | — | — |
 | AGit (no-fork) | — | ✅ | — | — | — |
-| CI status | — | ✅ | — | — | — |
+| CI status | ✅ | ✅ | — | — | — |
 | Templates | — | ✅ | — | — | — |
 | Auto-merge | — | — | — | ✅ | — |
 
@@ -176,14 +176,14 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Branches | ✅ | — | — | ✅ | — |
 | Topics | ✅ | — | — | ✅ | — |
 | Languages | ✅ | — | — | — | — |
-| Hooks | — | — | — | ✅ | — |
+| Hooks | ✅ | — | — | ✅ | — |
 | Archive | ✅ | — | — | ✅ | — |
 | Commits | ✅ | — | — | — | — |
 | Contents | ✅ | — | — | ✅ | — |
 | Collaborators | ✅ | — | — | ✅ | — |
 | Transfer | ✅ | — | — | ✅ | — |
-| Wikis | — | — | — | ✅ | — |
-| Push mirrors | — | — | — | ✅ | — |
+| Wikis | ✅ | — | — | ✅ | — |
+| Push mirrors | ✅ | — | — | ✅ | — |
 | Search | ✅ | — | — | ✅ | — |
 
 ### Labels
@@ -207,17 +207,17 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | | **cb** | **fj** | **berg** | **tea** | **gcli** |
 |---|---|---|---|---|---|
 | List | ✅ | — | ✅ | ✅ | ✅ |
-| Mark read | — | — | — | ✅ | — |
-| Thread details | — | — | — | — | — |
-| Per-repo | — | — | — | ✅ | — |
+| Mark read | ✅ | — | — | ✅ | — |
+| Thread details | ✅ | — | — | — | — |
+| Per-repo | ✅ | — | — | ✅ | — |
 
 ### Extra
 | | **cb** | **fj** | **berg** | **tea** | **gcli** |
 |---|---|---|---|---|---|
 | Raw API | ✅ | — | ✅ | — | ✅ |
 | User profiles | ✅ | ✅ | — | — | — |
-| SSH keys | — | ✅ | — | ✅ | ✅ |
-| GPG keys | — | ✅ | — | — | — |
+| SSH keys | ✅ | ✅ | — | ✅ | ✅ |
+| GPG keys | ✅ | ✅ | — | — | — |
 | Org/team mgmt | ✅ | ✅ | — | ✅ | — |
 | Forgejo Actions | ✅ | ✅ | — | — | — |
 | Shell completions | ✅ (auto via xclif) | ✅ | ✅ | ✅ | — |

{codeberg_cli-0.4.2 → codeberg_cli-0.5.0}/README.md

@@ -68,7 +68,7 @@ cb config set base_url "https://git.example.com/api/v1"  # Self-hosted Forgejo
 
 ## Comparison
 
-Here's how `cb` stacks up against other Forgejo CLI tools. *Last updated: May 2026.*
+Here's how `cb` stacks up against other Forgejo CLI tools. *Last updated: June 2026.*
 
 TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 
@@ -91,12 +91,12 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Delete | ✅ | — | — | — | — |
 | Pin/Unpin | ✅ | — | — | ✅ | — |
 | Search | ✅ | ✅ | — | ✅ | — |
-| Attachments | — | — | — | ✅ | — |
+| Attachments | ✅ | — | — | ✅ | — |
 | Labels (manage on issue) | ✅ | — | — | ✅ | ✅ |
-| Reactions | — | — | — | — | — |
+| Reactions | ✅ | — | — | — | — |
 | Subscriptions | ✅ | — | — | ✅ | — |
 | Tracked times | ✅ | — | — | ✅ | ✅ |
-| Dependencies | — | — | — | — | — |
+| Dependencies | ✅ | — | — | — | — |
 | Deadline | — | — | — | ✅ | — |
 | Templates | — | ✅ | — | — | — |
 
@@ -114,11 +114,11 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Checkout | ✅ | ✅ | ✅ | ✅ | ✅ |
 | Commits | ✅ | ✅ | — | — | ✅ |
 | Files/changed | ✅ | ✅ | — | — | — |
-| Reviews | — | ✅ | — | ✅ | ✅ |
+| Reviews | ✅ | ✅ | — | ✅ | ✅ |
 | Diff/Patch | ✅ | ✅ | — | — | ✅ |
 | Update branch | ✅ | — | — | — | — |
 | AGit (no-fork) | — | ✅ | — | — | — |
-| CI status | — | ✅ | — | — | — |
+| CI status | ✅ | ✅ | — | — | — |
 | Templates | — | ✅ | — | — | — |
 | Auto-merge | — | — | — | ✅ | — |
 
@@ -151,14 +151,14 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | Branches | ✅ | — | — | ✅ | — |
 | Topics | ✅ | — | — | ✅ | — |
 | Languages | ✅ | — | — | — | — |
-| Hooks | — | — | — | ✅ | — |
+| Hooks | ✅ | — | — | ✅ | — |
 | Archive | ✅ | — | — | ✅ | — |
 | Commits | ✅ | — | — | — | — |
 | Contents | ✅ | — | — | ✅ | — |
 | Collaborators | ✅ | — | — | ✅ | — |
 | Transfer | ✅ | — | — | ✅ | — |
-| Wikis | — | — | — | ✅ | — |
-| Push mirrors | — | — | — | ✅ | — |
+| Wikis | ✅ | — | — | ✅ | — |
+| Push mirrors | ✅ | — | — | ✅ | — |
 | Search | ✅ | — | — | ✅ | — |
 
 ### Labels
@@ -182,17 +182,17 @@ TL;DR: **`cb` is the most feature-complete Forgejo CLI** — by a wide margin.
 | | **cb** | **fj** | **berg** | **tea** | **gcli** |
 |---|---|---|---|---|---|
 | List | ✅ | — | ✅ | ✅ | ✅ |
-| Mark read | — | — | — | ✅ | — |
-| Thread details | — | — | — | — | — |
-| Per-repo | — | — | — | ✅ | — |
+| Mark read | ✅ | — | — | ✅ | — |
+| Thread details | ✅ | — | — | — | — |
+| Per-repo | ✅ | — | — | ✅ | — |
 
 ### Extra
 | | **cb** | **fj** | **berg** | **tea** | **gcli** |
 |---|---|---|---|---|---|
 | Raw API | ✅ | — | ✅ | — | ✅ |
 | User profiles | ✅ | ✅ | — | — | — |
-| SSH keys | — | ✅ | — | ✅ | ✅ |
-| GPG keys | — | ✅ | — | — | — |
+| SSH keys | ✅ | ✅ | — | ✅ | ✅ |
+| GPG keys | ✅ | ✅ | — | — | — |
 | Org/team mgmt | ✅ | ✅ | — | ✅ | — |
 | Forgejo Actions | ✅ | ✅ | — | — | — |
 | Shell completions | ✅ (auto via xclif) | ✅ | ✅ | ✅ | — |

{codeberg_cli-0.4.2 → codeberg_cli-0.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codeberg-cli"
-version = "0.4.2"
+version = "0.5.0"
 description = "A Forgejo CLI — works with Codeberg and any Forgejo instance"
 readme = "README.md"
 license = "MIT"