phabfive 0.6.0__tar.gz → 0.7.1__tar.gz

phabfive-0.7.1/.github/workflows/release.yml

@@ -0,0 +1,191 @@
+---
+name: Release
+
+on:
+  push:
+    tags:
+      - "v*"
+
+concurrency:
+  group: release-${{ github.ref }}
+  cancel-in-progress: false
+
+jobs:
+  build:
+    name: Build distribution
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v6
+        with:
+          enable-cache: true
+
+      - name: Set up Python
+        run: uv python install 3.13
+
+      - name: Build package
+        run: uv build
+
+      - name: Upload distribution artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: python-package-distributions
+          path: dist/
+
+  build-executables:
+    name: Build ${{ matrix.os }} ${{ matrix.arch }} executable
+    runs-on: ${{ matrix.runner }}
+    permissions:
+      id-token: write
+      contents: read
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          # AMD64 builds
+          - os: linux
+            arch: amd64
+            runner: ubuntu-latest
+            artifact_name: phabfive
+            asset_name: phabfive-linux-amd64
+          - os: macos
+            arch: amd64
+            runner: macos-14
+            artifact_name: phabfive
+            asset_name: phabfive-macos-amd64
+          - os: windows
+            arch: amd64
+            runner: windows-latest
+            artifact_name: phabfive.exe
+            asset_name: phabfive-windows-amd64.exe
+          # ARM64 builds
+          - os: linux
+            arch: arm64
+            runner: ubuntu-24.04-arm
+            artifact_name: phabfive
+            asset_name: phabfive-linux-arm64
+          - os: macos
+            arch: arm64
+            runner: macos-latest
+            artifact_name: phabfive
+            asset_name: phabfive-macos-arm64
+          - os: windows
+            arch: arm64
+            runner: windows-11-arm
+            artifact_name: phabfive.exe
+            asset_name: phabfive-windows-arm64.exe
+            skip_signing: true
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install Cosign
+        if: ${{ !matrix.skip_signing }}
+        uses: sigstore/cosign-installer@v3
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install pyinstaller
+          pip install .
+
+      - name: Build executable
+        shell: bash
+        run: |
+          pyinstaller --onefile --name phabfive phabfive/cli.py \
+            --collect-data rich \
+            --copy-metadata rich \
+            --collect-data phabricator \
+            --hidden-import phabricator \
+            --hidden-import anyconfig.backend.yaml \
+            --exclude-module mkdocs
+
+      - name: Rename artifact
+        shell: bash
+        run: mv dist/${{ matrix.artifact_name }} dist/${{ matrix.asset_name }}
+
+      - name: Sign executable with Sigstore
+        if: ${{ !matrix.skip_signing }}
+        shell: bash
+        run: |
+          cosign sign-blob dist/${{ matrix.asset_name }} \
+            --bundle dist/${{ matrix.asset_name }}.sigstore.json \
+            --yes
+
+      - name: Upload executable and signature
+        if: ${{ !matrix.skip_signing }}
+        uses: actions/upload-artifact@v4
+        with:
+          name: ${{ matrix.asset_name }}
+          path: |
+            dist/${{ matrix.asset_name }}
+            dist/${{ matrix.asset_name }}.sigstore.json
+
+      - name: Upload executable (unsigned)
+        if: ${{ matrix.skip_signing }}
+        uses: actions/upload-artifact@v4
+        with:
+          name: ${{ matrix.asset_name }}
+          path: dist/${{ matrix.asset_name }}
+
+  publish-pypi:
+    name: Publish to PyPI
+    needs: build
+    if: ${{ !contains(github.ref_name, '-rc') }}
+    runs-on: ubuntu-latest
+    environment:
+      name: pypi
+      url: https://pypi.org/p/phabfive
+    permissions:
+      id-token: write
+    steps:
+      - name: Download distribution artifacts
+        uses: actions/download-artifact@v4
+        with:
+          name: python-package-distributions
+          path: dist/
+
+      - name: Publish to PyPI
+        uses: pypa/gh-action-pypi-publish@release/v1
+
+  github-release:
+    name: Create GitHub Release
+    needs: [build, publish-pypi, build-executables]
+    if: ${{ always() && needs.build.result == 'success' && needs.build-executables.result == 'success' && (needs.publish-pypi.result == 'success' || needs.publish-pypi.result == 'skipped') }}
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Download all artifacts
+        uses: actions/download-artifact@v4
+        with:
+          path: artifacts/
+
+      - name: Prepare release assets
+        run: |
+          mkdir -p release-assets
+          # Copy Python distributions
+          cp artifacts/python-package-distributions/* release-assets/
+          # Copy executables
+          for dir in artifacts/phabfive-*/; do
+            cp "$dir"* release-assets/ 2>/dev/null || true
+          done
+          ls -la release-assets/
+
+      - name: Create GitHub Release
+        uses: softprops/action-gh-release@v2
+        with:
+          generate_release_notes: true
+          files: |
+            release-assets/*

phabfive-0.7.1/AGENTS.md

@@ -0,0 +1,89 @@
+# AGENTS.md
+
+This file provides guidance to AI coding agents when working with code in this repository.
+
+## Project Overview
+
+phabfive is a CLI for Phabricator and Phorge. It provides commands for interacting with Passphrase, Diffusion, Paste, User, and Maniphest applications.
+
+## Common Commands
+
+```bash
+# Install dependencies
+uv sync --group dev
+
+# Run the CLI
+uv run phabfive --help
+
+# Run tests
+uv run pytest                        # quick test with current Python
+uv run pytest tests/test_foo.py      # run single test file
+uv run pytest -k test_name           # run specific test
+uv run tox                           # test all Python versions (3.10-3.14)
+
+# Lint
+uv run flake8 phabfive/ tests/
+
+# Local Phorge instance for testing
+make up                              # start local Phorge
+make down                            # stop containers
+
+# Merge PRs (rebase only - merge and squash are disabled)
+gh pr merge --rebase
+```
+
+## Architecture
+
+### CLI Layer (`cli.py`)
+- Uses `docopt-ng` for argument parsing with docstring-based CLI definitions
+- Monogram shortcuts allow direct access: `phabfive T123` expands to `phabfive maniphest show T123`
+- Entry point: `cli_entrypoint()` → `parse_cli()`
+
+### Core Layer (`core.py`)
+- `Phabfive` class: central configuration and API client management
+- Loads config from environment variables or `~/.config/phabfive.yaml`
+- Uses the `phabricator` library for Conduit API calls
+- Output formatting: supports `rich` (terminal), `yaml`, and `strict` (machine-readable) modes
+
+### Feature Modules
+Complex features use a consistent subpackage structure:
+- `diffusion/` and `maniphest/` follow this pattern:
+  - `core.py` - main class inheriting from `Phabfive`
+  - `fetchers.py` - API data fetching
+  - `resolvers.py` - ID/name resolution
+  - `formatters.py` - output formatting
+  - `validators.py` - input validation
+
+### Simpler Modules
+- `passphrase.py`, `paste.py`, `user.py` - single-file implementations
+- `transitions/` - state machine for task status/priority/column changes
+
+### Configuration
+Required: `PHAB_TOKEN` and `PHAB_URL` (via environment or config file)
+
+## Version Management
+
+Version is defined only in `pyproject.toml`. Access it via:
+```python
+from importlib.metadata import version
+version("phabfive")
+```
+
+## Release Workflow
+
+Releases are triggered by pushing a git tag matching `v*`:
+
+```bash
+git tag -a v0.7.0 -m "Release v0.7.0"
+git push origin v0.7.0
+```
+
+**Artifacts produced:**
+- Python wheel and sdist → PyPI
+- Standalone executables for 6 platforms → GitHub Releases:
+  - `phabfive-linux-amd64`, `phabfive-linux-arm64`
+  - `phabfive-macos-amd64`, `phabfive-macos-arm64`
+  - `phabfive-windows-amd64.exe`, `phabfive-windows-arm64.exe`
+- Sigstore signatures (`.sigstore.json`) for all executables except Windows ARM64
+
+**RC tags** (containing `-rc`) skip PyPI but still build executables and create GitHub releases.

{phabfive-0.6.0 → phabfive-0.7.1}/CHANGELOG.md

@@ -1,3 +1,40 @@
+# 0.7.1 (2026-03-07)
+
+## Bug Fixes
+
+* Fix standalone executables exiting silently without output
+
+
+# 0.7.0 (2026-03-07)
+
+## Prelude
+
+This release adds interactive first-run configuration, standalone executables for all major platforms, and improved developer experience with Sigstore-signed releases.
+
+## New Features
+
+### Interactive Setup
+* **First-run configuration** - New `phabfive user setup` command guides users through initial configuration with interactive prompts
+* **Secure token input** - API token input is masked with dots for security
+* **Smart reconfiguration** - Warns before overwriting existing working configuration
+
+### Release Artifacts
+* **Standalone executables** - Pre-built binaries for Linux, macOS, and Windows (AMD64 and ARM64)
+* **Sigstore signing** - All executables are cryptographically signed for verification
+* **RC tag support** - Release candidates skip PyPI for testing the release process
+
+## Bug Fixes
+
+* Fix Rich markup escaping for square brackets in user content
+* Fix project lookup failing when more than 100 projects exist
+
+## Other Notes
+
+* Version now sourced solely from `pyproject.toml` using `importlib.metadata`
+* Added `AGENTS.md` for AI coding agent guidance
+* Removed unused `__author__`, `__email__`, `__url__` module attributes
+
+
 # 0.6.0 (2026-03-07)
 
 ## Prelude

phabfive-0.7.1/CLAUDE.md

@@ -0,0 +1 @@
+AGENTS.md

{phabfive-0.6.0 → phabfive-0.7.1}/PKG-INFO

@@ -1,7 +1,7 @@
 Metadata-Version: 2.4
 Name: phabfive
-Version: 0.6.0
-Summary: A command line tool to interact with Phabricator
+Version: 0.7.1
+Summary: CLI for Phabricator and Phorge - built for humans and AI agents
 Project-URL: Homepage, https://github.com/dynamist/phabfive
 Project-URL: Download, https://github.com/dynamist/phabfive/releases
 Author-email: Rickard Eriksson <rickard@dynamist.se>
@@ -41,7 +41,7 @@ Description-Content-Type: text/markdown
 
 # phabfive
 
-A command line tool to interact with [Phabricator](https://www.phacility.com/phabricator/) and [Phorge](https://www.phorge.it/).
+CLI for [Phabricator](https://www.phacility.com/phabricator/) and [Phorge](https://we.phorge.it/) - built for humans and AI agents.
 
 ![phabfive maniphest show](docs/maniphest-show.png)
 

{phabfive-0.6.0 → phabfive-0.7.1}/README.md

@@ -1,6 +1,6 @@
 # phabfive
 
-A command line tool to interact with [Phabricator](https://www.phacility.com/phabricator/) and [Phorge](https://www.phorge.it/).
+CLI for [Phabricator](https://www.phacility.com/phabricator/) and [Phorge](https://we.phorge.it/) - built for humans and AI agents.
 
 ![phabfive maniphest show](docs/maniphest-show.png)
 

{phabfive-0.6.0 → phabfive-0.7.1}/docs/releasing.md

@@ -77,9 +77,25 @@ git push origin v0.6.0
 
 This triggers the GitHub Actions workflow which will:
 
-1. Build the package
-2. Publish to PyPI using trusted publishing
-3. Create GitHub Release with auto-generated notes and artifacts
+1. Build the Python package (wheel and sdist)
+2. Build standalone executables for 6 platforms:
+   - Linux (AMD64, ARM64)
+   - macOS (AMD64, ARM64)
+   - Windows (AMD64, ARM64)
+3. Sign executables with [Sigstore](https://www.sigstore.dev/) (keyless OIDC signing)
+4. Publish to PyPI using trusted publishing
+5. Create GitHub Release with auto-generated notes and all artifacts
+
+**Testing with RC tags:** Tags containing `-rc` (e.g., `v0.7.0-rc.1`) will skip PyPI publishing but still build executables and create a GitHub Release. Useful for testing the release process.
+
+**Verifying signatures:** Users can verify downloaded executables with [cosign](https://docs.sigstore.dev/):
+
+```bash
+cosign verify-blob phabfive-linux-amd64 \
+  --bundle phabfive-linux-amd64.sigstore.json \
+  --certificate-identity-regexp="https://github.com/dynamist/phabfive" \
+  --certificate-oidc-issuer="https://token.actions.githubusercontent.com"
+```
 
 **5. Bump to Dev Version**
 

{phabfive-0.6.0 → phabfive-0.7.1}/phabfive/__init__.py

@@ -5,11 +5,6 @@ import logging
 import logging.config
 import sys
 
-__author__ = "Rickard Eriksson"
-__email__ = "rickard@dynamist.se"
-__version__ = "0.4.0"
-__url__ = "https://github.com/dynamist/phabfive"
-
 
 def init_logging(log_level):
     """

{phabfive-0.6.0 → phabfive-0.7.1}/phabfive/cli.py

@@ -5,10 +5,12 @@ import logging
 import os
 import re
 import sys
+from importlib.metadata import version
 from io import StringIO
 
 # 3rd party imports
 from docopt import DocoptExit, Option, docopt, extras
+from rich.markup import escape
 from rich.text import Text
 from rich.tree import Tree
 from ruamel.yaml import YAML
@@ -25,6 +27,32 @@ log = logging.getLogger(__name__)
 # =============================================================================
 
 
+def _escape_for_rich(content):
+    """Escape user content for safe Rich printing.
+
+    Rich interprets square brackets [...] as markup syntax.
+    This causes issues with user content containing:
+    - [/path] patterns -> MarkupError crash (interpreted as closing tag)
+    - [bug] tags -> Silently stripped (interpreted as invalid style)
+
+    Parameters
+    ----------
+    content : any
+        User-provided content to escape
+
+    Returns
+    -------
+    str or Text
+        Empty string for None, Text objects unchanged, escaped string otherwise
+    """
+    if content is None:
+        return ""
+    if isinstance(content, Text):
+        # Text objects are already safe Rich objects (e.g., hyperlinks)
+        return content
+    return escape(str(content))
+
+
 def _needs_yaml_quoting(value):
     """Check if a string value needs YAML quoting.
 
@@ -71,12 +99,12 @@ def display_task_rich(console, task_dict, phabfive_instance):
             # Multi-line value
             console.print(f"    {key}: |-")
             for line in str(value).splitlines():
-                console.print(f"      {line}")
+                console.print(f"      {_escape_for_rich(line)}")
         elif _needs_yaml_quoting(value):
-            escaped = value.replace("'", "''")
-            console.print(f"    {key}: '{escaped}'")
+            escaped = str(value).replace("'", "''")
+            console.print(f"    {key}: '{_escape_for_rich(escaped)}'")
         else:
-            console.print(f"    {key}: {value}")
+            console.print(f"    {key}: {_escape_for_rich(value)}")
 
     # Print Assignee
     if assignee:
@@ -126,10 +154,10 @@ def display_task_rich(console, task_dict, phabfive_instance):
                                 )
                             continue
                     if _needs_yaml_quoting(value):
-                        escaped = value.replace("'", "''")
-                        console.print(f"      {key}: '{escaped}'")
+                        escaped = str(value).replace("'", "''")
+                        console.print(f"      {key}: '{_escape_for_rich(escaped)}'")
                     else:
-                        console.print(f"      {key}: {value}")
+                        console.print(f"      {key}: {_escape_for_rich(value)}")
 
     # Print History section
     if history:
@@ -138,13 +166,13 @@ def display_task_rich(console, task_dict, phabfive_instance):
             if hist_key == "Boards" and isinstance(hist_value, dict):
                 console.print("    Boards:")
                 for board_name, transitions in hist_value.items():
-                    console.print(f"      {board_name}:")
+                    console.print(f"      {_escape_for_rich(board_name)}:")
                     for trans in transitions:
-                        console.print(f"        - {trans}")
+                        console.print(f"        - {_escape_for_rich(trans)}")
             elif isinstance(hist_value, list):
                 console.print(f"    {hist_key}:")
                 for trans in hist_value:
-                    console.print(f"      - {trans}")
+                    console.print(f"      - {_escape_for_rich(trans)}")
 
     # Print Comments section
     comments = task_dict.get("Comments", [])
@@ -154,11 +182,11 @@ def display_task_rich(console, task_dict, phabfive_instance):
             if isinstance(comment, PreservedScalarString) or "\n" in str(comment):
                 # Multi-line comment
                 lines = str(comment).splitlines()
-                console.print(f"    - {lines[0]}")
+                console.print(f"    - {_escape_for_rich(lines[0])}")
                 for line in lines[1:]:
-                    console.print(f"      {line}")
+                    console.print(f"      {_escape_for_rich(line)}")
             else:
-                console.print(f"    - {comment}")
+                console.print(f"    - {_escape_for_rich(comment)}")
 
     # Print Metadata section
     if metadata:
@@ -168,11 +196,11 @@ def display_task_rich(console, task_dict, phabfive_instance):
                 if meta_value:
                     console.print(f"    {meta_key}:")
                     for item in meta_value:
-                        console.print(f"      - {item}")
+                        console.print(f"      - {_escape_for_rich(item)}")
                 else:
                     console.print(f"    {meta_key}: []")
             else:
-                console.print(f"    {meta_key}: {meta_value}")
+                console.print(f"    {meta_key}: {_escape_for_rich(meta_value)}")
 
 
 def display_task_tree(console, task_dict, phabfive_instance):
@@ -207,9 +235,9 @@ def display_task_tree(console, task_dict, phabfive_instance):
             first_line = str(value).split("\n")[0]
             if len(first_line) > 60:
                 first_line = first_line[:57] + "..."
-            task_branch.add(f"{key}: {first_line}")
+            task_branch.add(f"{key}: {_escape_for_rich(first_line)}")
         else:
-            task_branch.add(f"{key}: {value}")
+            task_branch.add(f"{key}: {_escape_for_rich(value)}")
 
     # Add Assignee
     if assignee:
@@ -243,7 +271,7 @@ def display_task_tree(console, task_dict, phabfive_instance):
                             )
                             board_branch.add(Text.assemble("Column: ", column_link))
                             continue
-                    board_branch.add(f"{key}: {value}")
+                    board_branch.add(f"{key}: {_escape_for_rich(value)}")
 
     # Add History section
     if history:
@@ -252,13 +280,13 @@ def display_task_tree(console, task_dict, phabfive_instance):
             if hist_key == "Boards" and isinstance(hist_value, dict):
                 boards_hist = history_branch.add("Boards")
                 for board_name, transitions in hist_value.items():
-                    board_hist = boards_hist.add(board_name)
+                    board_hist = boards_hist.add(_escape_for_rich(board_name))
                     for trans in transitions:
-                        board_hist.add(trans)
+                        board_hist.add(_escape_for_rich(trans))
             elif isinstance(hist_value, list):
                 hist_type_branch = history_branch.add(hist_key)
                 for trans in hist_value:
-                    hist_type_branch.add(trans)
+                    hist_type_branch.add(_escape_for_rich(trans))
 
     # Add Comments section
     comments = task_dict.get("Comments", [])
@@ -270,9 +298,9 @@ def display_task_tree(console, task_dict, phabfive_instance):
                 first_line = str(comment).split("\n")[0]
                 if len(first_line) > 60:
                     first_line = first_line[:57] + "..."
-                comments_branch.add(first_line)
+                comments_branch.add(_escape_for_rich(first_line))
             else:
-                comments_branch.add(str(comment))
+                comments_branch.add(_escape_for_rich(str(comment)))
 
     # Add Metadata section
     if metadata:
@@ -282,11 +310,11 @@ def display_task_tree(console, task_dict, phabfive_instance):
                 if meta_value:
                     list_branch = meta_branch.add(meta_key)
                     for item in meta_value:
-                        list_branch.add(str(item))
+                        list_branch.add(_escape_for_rich(str(item)))
                 else:
                     meta_branch.add(f"{meta_key}: []")
             else:
-                meta_branch.add(f"{meta_key}: {meta_value}")
+                meta_branch.add(f"{meta_key}: {_escape_for_rich(meta_value)}")
 
     console.print(tree)
 
@@ -408,7 +436,7 @@ Available phabfive commands are:
     maniphest    The maniphest app
     paste        The paste app
     repl         Enter a REPL with API access
-    user         Information on users
+    user         Information on users, setup wizard
 
 Shortcuts to view Phabricator monograms (example: phabfive T123):
 
@@ -489,9 +517,12 @@ Paste Create Options:
 sub_user_args = """
 Usage:
     phabfive user whoami [options]
+    phabfive user setup [options]
 
 Options:
     -h, --help   Show this help message and exit
+
+The setup command configures phabfive with your Phabricator URL and API token.
 """
 
 sub_repl_args = """
@@ -681,17 +712,19 @@ def parse_cli():
     """
     import phabfive
 
+    pkg_version = version("phabfive")
+
     try:
         cli_args = docopt(
             base_args,
             options_first=True,
-            version=phabfive.__version__,
+            version=pkg_version,
             default_help=True,
         )
     except DocoptExit:
         extras(
             True,
-            phabfive.__version__,
+            pkg_version,
             [Option("-h", "--help", 0, True)],
             base_args,
         )
@@ -778,7 +811,7 @@ def parse_cli():
     else:
         extras(
             True,
-            phabfive.__version__,
+            pkg_version,
             [Option("-h", "--help", 0, True)],
             base_args,
         )
@@ -798,7 +831,7 @@ def run(cli_args, sub_args):
     from phabfive import diffusion, maniphest, passphrase, paste, repl, user
     from phabfive.constants import REPO_STATUS_CHOICES
     from phabfive.core import Phabfive
-    from phabfive.exceptions import PhabfiveException
+    from phabfive.exceptions import PhabfiveConfigException, PhabfiveException
     from phabfive.transitions import parse_column_patterns, parse_priority_patterns
 
     # Validate and process output options
@@ -961,6 +994,13 @@ def run(cli_args, sub_args):
                         print(f"{p['id']} {p['title']}")
 
         if cli_args["<command>"] == "user":
+            # Handle setup first - it doesn't require existing configuration
+            if sub_args.get("setup"):
+                from phabfive.setup import SetupWizard
+
+                wizard = SetupWizard()
+                return 0 if wizard.run() else 1
+
             user_app = user.User()
 
             if sub_args["whoami"]:
@@ -1232,6 +1272,14 @@ def run(cli_args, sub_args):
                     show_comments=show_comments,
                 )
                 display_tasks(result, output_format, maniphest_app)
+    except PhabfiveConfigException as e:
+        # Offer interactive setup for configuration errors
+        from phabfive.setup import offer_setup_on_error
+
+        if offer_setup_on_error(str(e)):
+            # Setup succeeded - connection verified
+            return 0
+        retcode = 1
     except PhabfiveException as e:
         # Catch all types of phabricator base exceptions
         log.critical(str(e))
@@ -1264,3 +1312,7 @@ def cli_entrypoint():
         sys.exit(130)  # Standard Unix exit code for SIGINT (128 + 2)
     except Exception:
         raise
+
+
+if __name__ == "__main__":
+    cli_entrypoint()