refactor(wizard): replace hand-rolled arrow-key menu with questionary

The reviewer asked for a known-good interactive-menu library instead of
the custom termios/select key reader; arrow keys had failed in their
terminal. The TTY path now delegates to questionary.select (pointer ">",
bold highlight via pointer/selected styles), keeping choice values mapped
to list indices. Cancellable menus graft an eager Escape binding onto the
prompt's own key-binding registry so Esc backs out while Ctrl-C still
aborts fixed menus and backs out of cancellable ones. The non-TTY
numbered fallback is untouched, and hand-rolled internals
(_arrow_menu/_read_keypress/_option_line/_draw_menu_rows) are gone.

README.md

@@ -4,6 +4,17 @@ Aggregate weekly activity across development platforms (Launchpad, GitHub, GitLa
 
 ## Usage
 
+```bash
+# Interactive setup: pick sources, store identities/tokens once
+uv run weekly-activity config
+
+# Aggregated report across every configured source
+uv run weekly-activity report      # bare `weekly-activity` does the same
+```
+
+### Single sources (ad-hoc, no config needed)
+
+
 ```bash
 # Launchpad (uses system keyring for OAuth; --anonymous for public data only)
 uv run weekly-activity launchpad <username>
@@ -54,13 +65,68 @@ Any GitLab instance works via `--url`. Token discovery order:
 
 Rolling 7 days ending today (inclusive). Override with `--since` / `--until` (ISO dates, UTC).
 
+## Configuration
+
+`weekly-activity config` opens a small menu-driven editor: it lists your
+current sources (secrets masked), then lets you **Add a new source**
+(choose the kind, repeat as often as you like), **Remove a source**, or
+**Exit** (writes the file only when something changed). Several accounts
+of the same provider can live side by side. Tokens are stored as provided
+(plaintext), but the file is created with owner-only permissions (`0600`)
+and secrets are never echoed back. Use `--config <path>` (also accepted by
+`report`) to keep configs elsewhere.
+
+Default location: `${XDG_CONFIG_HOME:-~/.config}/weekly-activity.toml`.
+
+```toml
+version = 2
+
+[[launchpad.accounts]]
+username = "jane"
+mode = "credentials"   # "anonymous" (public data only) | "credentials"
+# credentials_file = "~/lp-creds.json"   # omit -> system keyring
+
+[[github.accounts]]
+name = "personal"      # optional display label
+username = "octocat"
+token = ""             # empty -> $token_env / GITHUB_TOKEN / `gh auth token`
+
+[[github.accounts]]
+name = "work"
+username = "acme-jane"
+token_env = "WORK_GH_TOKEN"
+
+[[gitlab.accounts]]
+name = "salsa"
+url = "https://salsa.debian.org"
+username = "jane"
+token_env = "SALSA_TOKEN"
+
+[[bts.accounts]]
+email = "jane@example.org"
+```
+
+An absent section disables that source; an empty one is not written.
+When a non-anonymous Launchpad account is added without a credentials
+file, the wizard runs launchpadlib's OAuth right there — a browser opens
+once and the token is kept in your system keyring, so `report` never has
+to authenticate.
+
+Legacy configs without the `version` key (one single-account section per
+source) keep working: they are migrated transparently when loaded and
+rewritten in the v2 shape the next time the wizard saves.
+
+
 ## Architecture
 
 ```
 src/weekly_activity/
   model.py            ActivityReport, Section, last_week_window
   report.py           format_report() — source-agnostic text renderer
-  cli.py              subcommand dispatch (launchpad | github | gitlab | bts)
+  cli.py              subcommand dispatch (launchpad | github | gitlab | bts | report | config)
+  aggregate.py        collect_specs()/format_combined() — one text report across sources
+  config.py           ActivityConfig schema + TOML load/save (tomllib read, hand-written)
+  wizard.py           interactive `config` onboarding
   sources/
     __init__.py       ActivitySource protocol
     launchpad.py      LaunchpadSource (launchpadlib)

pyproject.toml

@@ -8,6 +8,7 @@ dependencies = [
     "keyring>=25",
     "httpx>=0.27",
     "python-debianbts>=4.1.1",
+    "questionary>=2.1.1",
 ]
 
 [project.scripts]

src/weekly_activity/aggregate.py

@@ -0,0 +1,173 @@
+"""Combined report — build every configured account and render one text output.
+
+Each configured account runs independently: a failure (bad token,
+unreachable API, unknown user) is recorded as a ``[skipped: <source> …]``
+block instead of aborting the whole report. Per-source warnings surfaced by
+``collect`` are rendered inside the source's block by ``format_report``.
+"""
+
+from __future__ import annotations
+
+import os
+from collections.abc import Callable
+from dataclasses import dataclass, replace
+from datetime import datetime, timedelta
+
+from weekly_activity.config import ActivityConfig
+from weekly_activity.model import ActivityReport
+from weekly_activity.report import format_report
+from weekly_activity.sources import ActivitySource
+from weekly_activity.sources.debian_bts import DebianBtsSource
+from weekly_activity.sources.github import GitHubSource
+from weekly_activity.sources.gitlab import GitLabSource
+from weekly_activity.sources.launchpad import LaunchpadSource
+
+_RULE = "─" * 60
+
+
+@dataclass(frozen=True, slots=True)
+class SourceSpec:
+    """One configured account: display name, identity to query, deferred constructor.
+
+    ``name`` carries the report-heading label: the source kind plus the
+    account's ``name`` (or identity) whenever more than one account of that
+    kind exists and disambiguation helps.
+    """
+
+    name: str
+    username: str
+    make: Callable[[], ActivitySource]
+
+
+@dataclass(frozen=True, slots=True)
+class Collected:
+    spec: SourceSpec
+    report: ActivityReport | None = None
+    error: str | None = None
+
+
+def source_label(kind: str, name: str, identity: str, ambiguous: bool) -> str:
+    """Heading label for an account: the source kind, disambiguated when needed.
+
+    An explicit ``name`` label always shows; otherwise sibling accounts of
+    the same kind force the identity (username/email) into the label. Shared
+    with the wizard so menus and report headings never drift apart.
+    """
+    if name:
+        return f"{kind}/{name}"
+    if ambiguous and identity:
+        return f"{kind}/{identity}"
+    return kind
+
+
+def build_specs(config: ActivityConfig) -> list[SourceSpec]:
+    """Map every configured account onto a constructible source (network deferred).
+
+    Accounts appear in listing order within each source kind, kinds in
+    stable order: launchpad, github, gitlab, bts.
+    """
+    ambiguous = {
+        "launchpad": len(config.launchpad) > 1,
+        "github": len(config.github) > 1,
+        "gitlab": len(config.gitlab) > 1,
+        "bts": len(config.bts) > 1,
+    }
+    specs: list[SourceSpec] = []
+    for lp in config.launchpad:
+        specs.append(
+            SourceSpec(
+                name=source_label("launchpad", lp.name, lp.username, ambiguous["launchpad"]),
+                username=lp.username,
+                make=_make_launchpad(lp.service, lp.mode == "anonymous", lp.credentials_file),
+            )
+        )
+    for gh in config.github:
+        specs.append(
+            SourceSpec(
+                name=source_label("github", gh.name, gh.username, ambiguous["github"]),
+                username=gh.username,
+                make=_make_github(gh.token, gh.token_env),
+            )
+        )
+    for gl in config.gitlab:
+        specs.append(
+            SourceSpec(
+                name=source_label("gitlab", gl.name, gl.username, ambiguous["gitlab"]),
+                username=gl.username,
+                make=_make_gitlab(gl.url, gl.token, gl.token_env),
+            )
+        )
+    for bts_acct in config.bts:
+        email = bts_acct.email
+        specs.append(
+            SourceSpec(
+                name=source_label("bts", bts_acct.name, email, ambiguous["bts"]),
+                username=email,
+                make=DebianBtsSource,
+            )
+        )
+    return specs
+
+
+def _make_launchpad(
+    service: str, anonymous: bool, credentials_file: str
+) -> Callable[[], ActivitySource]:
+    expanded = os.path.expanduser(credentials_file)
+
+    def make() -> ActivitySource:
+        return LaunchpadSource(
+            service=service,
+            anonymous=anonymous,
+            credentials_file=expanded or None,
+        )
+
+    return make
+
+
+def _make_github(token: str, token_env: str) -> Callable[[], ActivitySource]:
+    def make() -> ActivitySource:
+        # No stored token: explicit env var if named, else GitHubSource's own
+        # GITHUB_TOKEN / `gh auth token` discovery.
+        resolved = token or (os.environ.get(token_env, "") if token_env else "")
+        return GitHubSource(token=resolved or None)
+
+    return make
+
+
+def _make_gitlab(url: str, token: str, token_env: str) -> Callable[[], ActivitySource]:
+    def make() -> ActivitySource:
+        return GitLabSource(url=url, token=token or None, token_env=token_env or None)
+
+    return make
+
+
+def collect_specs(specs: list[SourceSpec], since: datetime, until: datetime) -> list[Collected]:
+    """Run every spec, converting any per-source failure into an error record."""
+    collected: list[Collected] = []
+    for spec in specs:
+        try:
+            source = spec.make()
+            report = source.collect(spec.username, since, until)
+        except Exception as exc:  # noqa: BLE001 - one bad source must not kill the rest
+            collected.append(Collected(spec, error=f"{type(exc).__name__}: {exc}"))
+        else:
+            # Headings must show the account label, not just the source kind.
+            collected.append(Collected(spec, report=replace(report, source=spec.name)))
+    return collected
+
+
+def format_combined(collected: list[Collected], since: datetime, until: datetime) -> str:
+    """Render all blocks under one header; failed sources become skip notes."""
+    end_date = (until - timedelta(days=1)).strftime("%Y-%m-%d")
+    header = (
+        f"Weekly activity report — {len(collected)} source(s) — "
+        f"{since.strftime('%Y-%m-%d')} .. {end_date} (UTC, inclusive)"
+    )
+    blocks: list[str] = []
+    for item in collected:
+        if item.report is not None:
+            blocks.append(format_report(item.report))
+        else:
+            detail = item.error or "unknown error"
+            blocks.append(f"[skipped: {item.spec.name} — {detail}]")
+    return ("\n\n" + _RULE + "\n\n").join([header, *blocks])

src/weekly_activity/cli.py

@@ -6,13 +6,23 @@ import argparse
 import sys
 from collections.abc import Sequence
 from datetime import UTC, datetime
+from pathlib import Path
 
+from weekly_activity.aggregate import (
+    Collected,
+    SourceSpec,
+    build_specs,
+    collect_specs,
+    format_combined,
+)
+from weekly_activity.config import ConfigError, default_config_path, load_config
 from weekly_activity.model import last_week_window
 from weekly_activity.report import format_report
 from weekly_activity.sources.debian_bts import DebianBtsSource
 from weekly_activity.sources.github import GitHubSource
 from weekly_activity.sources.gitlab import GitLabSource
 from weekly_activity.sources.launchpad import LaunchpadSource
+from weekly_activity.wizard import run_wizard
 
 
 def parse_args(argv: Sequence[str] | None = None) -> argparse.Namespace:
@@ -26,13 +36,28 @@ def parse_args(argv: Sequence[str] | None = None) -> argparse.Namespace:
         help="Inclusive end as an ISO date/datetime (UTC). Default: today.",
     )
 
+    common = argparse.ArgumentParser(add_help=False)
+    common.add_argument(
+        "--config",
+        default=None,
+        metavar="PATH",
+        help="Config file location. Default: ~/.config/weekly-activity.toml.",
+    )
+
     parser = argparse.ArgumentParser(
         prog="weekly-activity",
-        description="Summarize a user's activity for last week (or a custom range).",
+        description=(
+            "Summarize dev-platform activity for a week - one source at a time,"
+            " or aggregated across everything configured."
+        ),
+    )
+    subparsers = parser.add_subparsers(
+        dest="command",
+        required=False,
+        help="a single source, `report` for all configured ones, `config` to set up",
     )
-    subparsers = parser.add_subparsers(dest="source", required=True)
 
-    lp = subparsers.add_parser("launchpad", parents=[base], help="Launchpad activity.")
+    lp = subparsers.add_parser("launchpad", parents=[base, common], help="Launchpad activity.")
     lp.add_argument("username", help="Launchpad username (the ~name).")
     lp.add_argument(
         "--anonymous",
@@ -46,13 +71,13 @@ def parse_args(argv: Sequence[str] | None = None) -> argparse.Namespace:
         default="production",
     )
 
-    gh = subparsers.add_parser("github", parents=[base], help="GitHub activity.")
+    gh = subparsers.add_parser("github", parents=[base, common], help="GitHub activity.")
     gh.add_argument("username", help="GitHub username.")
     gh.add_argument("--token", default=None, help="GitHub token (or set GITHUB_TOKEN env var).")
 
     gl = subparsers.add_parser(
         "gitlab",
-        parents=[base],
+        parents=[base, common],
         help="GitLab activity (gitlab.com or any self-hosted instance, e.g. Debian Salsa).",
     )
     gl.add_argument("username", help="GitLab username.")
@@ -69,9 +94,22 @@ def parse_args(argv: Sequence[str] | None = None) -> argparse.Namespace:
         help="Env var holding the API token (see README for discovery order).",
     )
 
-    bts = subparsers.add_parser("bts", parents=[base], help="Debian Bug Tracking System activity.")
+    bts = subparsers.add_parser(
+        "bts", parents=[base, common], help="Debian Bug Tracking System activity."
+    )
     bts.add_argument("username", help="Email address used on Debian bugs (submitter/owner).")
 
+    subparsers.add_parser(
+        "report",
+        parents=[base, common],
+        help="Aggregate activity from every source in the config file.",
+    )
+    subparsers.add_parser(
+        "config",
+        parents=[common],
+        help="Interactive wizard that writes the config file.",
+    )
+
     return parser.parse_args(argv)
 
 
@@ -96,24 +134,84 @@ def resolve_period(args: argparse.Namespace) -> tuple[datetime, datetime]:
     return last_week_window()
 
 
+_STATUS_PREFIX = "# Generating report"
+_STATUS_ERASE = "\x1b[2K\r"
+
+
+def _report_status_active() -> bool:
+    """Transient progress lines only make sense on an interactive terminal."""
+    return sys.stdout.isatty()
+
+
+def _show_report_status(name: str) -> None:
+    """Erase-then-overwrite in place; no newline keeps updates on one row."""
+    sys.stdout.write(f"{_STATUS_ERASE}{_STATUS_PREFIX}: pulling {name} data...")
+    sys.stdout.flush()
+
+
+def _hide_report_status() -> None:
+    """Erase the status row so nothing of it survives into the final output."""
+    sys.stdout.write(_STATUS_ERASE)
+    sys.stdout.flush()
+
+
+def _collect_report_sources(
+    specs: list[SourceSpec], since: datetime, until: datetime
+) -> list[Collected]:
+    """Run collect_specs per source, showing a TTY-only progress line each."""
+    show = _report_status_active()
+    collected: list[Collected] = []
+    for spec in specs:
+        if show:
+            _show_report_status(spec.name)
+        collected.extend(collect_specs([spec], since, until))
+    if show:
+        _hide_report_status()
+    return collected
+
+
 def main() -> None:
     args = parse_args()
+    command = args.command or "report"
+
+    if command == "config":
+        run_wizard(_config_path(args))
+        return
+
+    if command == "report":
+        path = _config_path(args)
+        try:
+            config = load_config(path)
+        except FileNotFoundError as exc:
+            raise SystemExit(
+                f"error: no configuration found at {path}; run `weekly-activity config` first."
+            ) from exc
+        except ConfigError as exc:
+            raise SystemExit(f"error: {exc}") from exc
+        specs = build_specs(config)
+        if not specs:
+            raise SystemExit(f"error: {path} enables no sources; run `weekly-activity config`.")
+        since, until = resolve_period(args)
+        collected = _collect_report_sources(specs, since, until)
+        print(format_combined(collected, since, until))
+        return
+
     since, until = resolve_period(args)
 
-    if args.source == "launchpad":
+    if command == "launchpad":
         source = LaunchpadSource(
             service=args.service,
             anonymous=args.anonymous,
             credentials_file=args.credentials_file,
         )
-    elif args.source == "github":
+    elif command == "github":
         source = GitHubSource(token=args.token)
-    elif args.source == "gitlab":
+    elif command == "gitlab":
         source = GitLabSource(url=args.url, token=args.token, token_env=args.token_env)
-    elif args.source == "bts":
+    elif command == "bts":
         source = DebianBtsSource()
     else:
-        raise SystemExit(f"error: unknown source {args.source!r}")
+        raise SystemExit(f"error: unknown source {command!r}")
 
     try:
         report = source.collect(args.username, since, until)
@@ -126,5 +224,12 @@ def main() -> None:
     print(format_report(report))
 
 
+def _config_path(args: argparse.Namespace) -> Path:
+    """Resolve this run's config location (--config override or XDG default)."""
+    if args.config:
+        return Path(args.config).expanduser()
+    return default_config_path()
+
+
 if __name__ == "__main__":
     main()

src/weekly_activity/config.py

@@ -0,0 +1,386 @@
+"""Config file: schema, discovery, loading, migration, and persistence.
+
+Stored as TOML at ``${XDG_CONFIG_HOME:-~/.config}/weekly-activity.toml``
+(override with ``--config``).
+
+Version 2 layout (``version = 2``): one section per source kind, each
+holding an array of accounts (``[[github.accounts]]``, …) so several
+accounts or instances of the same kind can be configured side by side.
+An absent section means that source is not enabled.
+
+Legacy v1 files (no ``version`` key, one single-account section per
+source) are transparently migrated to the account-list shape when loaded,
+in memory only — the file on disk is never rewritten by ``load_config``;
+the next save from the wizard persists the migrated v2 shape.
+
+Tokens are stored as provided (plaintext) in the user's own local config;
+the file is created with ``0600`` permissions so they are not world-readable.
+"""
+
+from __future__ import annotations
+
+import os
+import tomllib
+from collections.abc import Callable
+from dataclasses import dataclass, field
+from pathlib import Path
+from typing import TypeVar
+
+CONFIG_FILE_NAME = "weekly-activity.toml"
+CONFIG_VERSION = 2
+
+_SOURCE_KEYS = ("launchpad", "github", "gitlab", "bts")
+_LAUNCHPAD_MODES = frozenset({"anonymous", "credentials"})
+_LAUNCHPAD_SERVICES = frozenset({"production", "staging"})
+
+_T = TypeVar("_T")
+
+
+class ConfigError(Exception):
+    """The config file exists but is malformed or has unknown entries."""
+
+
+@dataclass(slots=True)
+class GithubSettings:
+    """One GitHub account."""
+
+    username: str
+    token: str = ""  # empty -> $token_env / GITHUB_TOKEN / `gh auth token`
+    token_env: str = ""  # optional named var, takes precedence in discovery
+    name: str = ""  # optional display label
+
+
+@dataclass(slots=True)
+class LaunchpadSettings:
+    """One Launchpad account.
+
+    In credentials mode an empty ``credentials_file`` means launchpadlib
+    uses its system-keyring credential store; config-time OAuth in the
+    wizard populates that keyring so ``report`` never needs to authenticate.
+    """
+
+    username: str
+    mode: str = "anonymous"  # "anonymous" | "credentials"
+    credentials_file: str = ""  # used in "credentials" mode; empty -> keyring
+    service: str = "production"
+    name: str = ""
+
+
+@dataclass(slots=True)
+class GitlabSettings:
+    """One GitLab instance/account."""
+
+    username: str
+    url: str = "https://gitlab.com"
+    token: str = ""  # empty -> env discovery (<HOST>_TOKEN, GITLAB_TOKEN)
+    token_env: str = ""  # optional named var, takes precedence in discovery
+    name: str = ""
+
+
+@dataclass(slots=True)
+class BtsSettings:
+    """One Debian BTS identity (an email address)."""
+
+    email: str
+    name: str = ""
+
+
+@dataclass(slots=True)
+class ActivityConfig:
+    """All configured accounts; an empty list = source disabled."""
+
+    launchpad: list[LaunchpadSettings] = field(default_factory=list)
+    github: list[GithubSettings] = field(default_factory=list)
+    gitlab: list[GitlabSettings] = field(default_factory=list)
+    bts: list[BtsSettings] = field(default_factory=list)
+
+    def enabled_sources(self) -> list[str]:
+        """Names of configured source sections, in stable report order."""
+        ordered = (
+            ("launchpad", self.launchpad),
+            ("github", self.github),
+            ("gitlab", self.gitlab),
+            ("bts", self.bts),
+        )
+        return [name for name, accounts in ordered if accounts]
+
+
+def default_config_path() -> Path:
+    xdg = os.environ.get("XDG_CONFIG_HOME")
+    base = Path(xdg) if xdg else Path.home() / ".config"
+    return base / CONFIG_FILE_NAME
+
+
+# --- Loading ---
+
+
+def load_config(path: Path) -> ActivityConfig:
+    """Parse the TOML config at ``path``.
+
+    Files written at ``version = 2`` carry arrays of accounts per source.
+    Legacy files without a ``version`` key are migrated to that shape in
+    memory (one unnamed account per configured section); nothing is
+    rewritten on disk here — saving from the wizard persists v2.
+
+    Raises ``FileNotFoundError`` when the file does not exist and
+    ``ConfigError`` when it cannot be parsed or contains unknown entries.
+    """
+    try:
+        text = path.read_text(encoding="utf-8")
+    except FileNotFoundError:
+        raise
+    except OSError as exc:
+        raise ConfigError(f"cannot read {path}: {exc}") from exc
+    try:
+        data = tomllib.loads(text)
+    except tomllib.TOMLDecodeError as exc:
+        raise ConfigError(f"{path}: invalid TOML: {exc}") from exc
+
+    where = str(path)
+    _reject_unknown_top_level(data, where)
+
+    if "version" in data:
+        _check_version(data["version"], where)
+        return ActivityConfig(
+            launchpad=_account_list(data.get("launchpad"), "launchpad", _parse_launchpad),
+            github=_account_list(data.get("github"), "github", _parse_github),
+            gitlab=_account_list(data.get("gitlab"), "gitlab", _parse_gitlab),
+            bts=_account_list(data.get("bts"), "bts", _parse_bts),
+        )
+
+    if _looks_like_v2_without_version(data):
+        raise ConfigError(
+            f"{where}: account-array config without a version header; "
+            f'add "version = {CONFIG_VERSION}" at the top'
+        )
+    # Legacy v1 layout: migrate one single-table section to one unnamed account.
+    return ActivityConfig(
+        launchpad=_legacy_accounts(data.get("launchpad"), "launchpad", _parse_launchpad),
+        github=_legacy_accounts(data.get("github"), "github", _parse_github),
+        gitlab=_legacy_accounts(data.get("gitlab"), "gitlab", _parse_gitlab),
+        bts=_legacy_accounts(data.get("bts"), "bts", _parse_bts),
+    )
+
+
+def _check_version(version: object, where: str) -> None:
+    if isinstance(version, bool) or not isinstance(version, int):
+        raise ConfigError(f"{where}: version must be an integer")
+    if version != CONFIG_VERSION:
+        raise ConfigError(
+            f"{where}: unsupported config version {version} "
+            f"(this release understands version {CONFIG_VERSION})"
+        )
+
+
+def _reject_unknown_top_level(data: dict[str, object], where: str) -> None:
+    unknown = sorted(set(data) - {"version", *_SOURCE_KEYS})
+    if unknown:
+        raise ConfigError(f"{where}: unknown config section(s): {', '.join(unknown)}")
+
+
+def _looks_like_v2_without_version(data: dict[str, object]) -> bool:
+    for key in _SOURCE_KEYS:
+        section = data.get(key)
+        if isinstance(section, dict) and "accounts" in section:
+            return True
+    return False
+
+
+def _account_list(raw: object, section: str, parse: Callable[[object, str], _T]) -> list[_T]:
+    """Parse ``[[<section>.accounts]]`` tables via the given account parser."""
+    if raw is None:
+        return []
+    table = _as_table(raw, section)
+    _check_keys(table, {"accounts"}, section)
+    accounts = table["accounts"]
+    if not isinstance(accounts, list):
+        raise ConfigError(
+            f"[{section}] accounts must be an array of tables ([[{section}.accounts]])"
+        )
+    return [parse(item, f"{section}.accounts[{index}]") for index, item in enumerate(accounts)]
+
+
+def _legacy_accounts(raw: object, section: str, parse: Callable[[object, str], _T]) -> list[_T]:
+    """Migrate one v1 single-table section into a single-account list."""
+    if raw is None:
+        return []
+    return [parse(raw, section)]
+
+
+def _check_keys(table: dict[str, object], names: set[str], section: str) -> None:
+    unknown = sorted(set(table) - names)
+    if unknown:
+        raise ConfigError(f"[{section}]: unknown key(s): {', '.join(unknown)}")
+
+
+def _required_str(table: dict[str, object], key: str, section: str) -> str:
+    value = table[key]
+    if not isinstance(value, str):
+        raise ConfigError(f"[{section}] {key} must be a string")
+    return value
+
+
+def _optional_str(table: dict[str, object], key: str, section: str) -> str:
+    value = table[key]
+    if not isinstance(value, str):
+        raise ConfigError(f"[{section}] {key} must be a string")
+    return value
+
+
+def _optional_default_str(table: dict[str, object], key: str, section: str) -> str:
+    return _optional_str(table, key, section) if key in table else ""
+
+
+def _one_of(value: str, allowed: frozenset[str], section: str, key: str) -> str:
+    if value not in allowed:
+        choices = ", ".join(sorted(allowed))
+        raise ConfigError(f"[{section}] {key} must be one of: {choices}")
+    return value
+
+
+def _parse_github(raw: object, section: str) -> GithubSettings:
+    table = _as_table(raw, section)
+    _check_keys(table, {"name", "username", "token", "token_env"}, section)
+    return GithubSettings(
+        username=_required_str(table, "username", section),
+        token=_optional_default_str(table, "token", section),
+        token_env=_optional_default_str(table, "token_env", section),
+        name=_optional_default_str(table, "name", section),
+    )
+
+
+def _parse_launchpad(raw: object, section: str) -> LaunchpadSettings:
+    table = _as_table(raw, section)
+    _check_keys(table, {"name", "username", "mode", "credentials_file", "service"}, section)
+    mode = _one_of(
+        _optional_default_str(table, "mode", section) or "anonymous",
+        _LAUNCHPAD_MODES,
+        section,
+        "mode",
+    )
+    service = _one_of(
+        _optional_default_str(table, "service", section) or "production",
+        _LAUNCHPAD_SERVICES,
+        section,
+        "service",
+    )
+    return LaunchpadSettings(
+        username=_required_str(table, "username", section),
+        mode=mode,
+        credentials_file=_optional_default_str(table, "credentials_file", section),
+        service=service,
+        name=_optional_default_str(table, "name", section),
+    )
+
+
+def _parse_gitlab(raw: object, section: str) -> GitlabSettings:
+    table = _as_table(raw, section)
+    _check_keys(table, {"name", "username", "url", "token", "token_env"}, section)
+    return GitlabSettings(
+        username=_required_str(table, "username", section),
+        url=_optional_default_str(table, "url", section) or "https://gitlab.com",
+        token=_optional_default_str(table, "token", section),
+        token_env=_optional_default_str(table, "token_env", section),
+        name=_optional_default_str(table, "name", section),
+    )
+
+
+def _parse_bts(raw: object, section: str) -> BtsSettings:
+    table = _as_table(raw, section)
+    _check_keys(table, {"name", "email"}, section)
+    return BtsSettings(
+        email=_required_str(table, "email", section),
+        name=_optional_default_str(table, "name", section),
+    )
+
+
+def _as_table(raw: object, section: str) -> dict[str, object]:
+    if not isinstance(raw, dict):
+        raise ConfigError(f"[{section}] must be a TOML table")
+    return raw
+
+
+# --- Saving (hand-formatted: the schema is flat and string-only) ---
+
+
+def save_config(config: ActivityConfig, path: Path) -> None:
+    """Serialize ``config`` to TOML at ``path`` with owner-only permissions."""
+    path.parent.mkdir(parents=True, exist_ok=True)
+    fd = os.open(path, os.O_WRONLY | os.O_CREAT | os.O_TRUNC, 0o600)
+    with os.fdopen(fd, "w", encoding="utf-8") as fh:
+        fh.write(format_toml(config))
+    os.chmod(path, 0o600)  # enforce even when overwriting a pre-existing file
+
+
+def format_toml(config: ActivityConfig) -> str:
+    lines = [
+        "# weekly-activity configuration.",
+        "# Generated by `weekly-activity config`; edit by hand if you prefer.",
+        "# Tokens are stored in plaintext and readable only by your user (0600).",
+        "# Run `weekly-activity report` to aggregate every enabled source below.",
+        "",
+        f"version = {CONFIG_VERSION}",
+    ]
+
+    def emit(section: str, entry_lines: list[str]) -> None:
+        lines.append("")
+        lines.append(f"[[{section}.accounts]]")
+        lines.extend(entry_lines)
+
+    for lp in config.launchpad:
+        entry = [f"name = {_toml(lp.name)}"] if lp.name else []
+        entry.append(f"username = {_toml(lp.username)}")
+        if lp.mode != "anonymous":
+            entry.append(f"mode = {_toml(lp.mode)}")
+        if lp.credentials_file:
+            entry.append(f"credentials_file = {_toml(lp.credentials_file)}")
+        if lp.service != "production":
+            entry.append(f"service = {_toml(lp.service)}")
+        emit("launchpad", entry)
+    for gh in config.github:
+        entry = [f"name = {_toml(gh.name)}"] if gh.name else []
+        entry.append(f"username = {_toml(gh.username)}")
+        if gh.token_env:
+            entry.append(f"token_env = {_toml(gh.token_env)}")
+        if gh.token:
+            entry.append(f"token = {_toml(gh.token)}")
+        emit("github", entry)
+    for gl in config.gitlab:
+        entry = [f"name = {_toml(gl.name)}"] if gl.name else []
+        entry.append(f"username = {_toml(gl.username)}")
+        if gl.url != "https://gitlab.com":
+            entry.append(f"url = {_toml(gl.url)}")
+        if gl.token:
+            entry.append(f"token = {_toml(gl.token)}")
+        if gl.token_env:
+            entry.append(f"token_env = {_toml(gl.token_env)}")
+        emit("gitlab", entry)
+    for bts in config.bts:
+        entry = [f"name = {_toml(bts.name)}"] if bts.name else []
+        entry.append(f"email = {_toml(bts.email)}")
+        emit("bts", entry)
+    return "\n".join(lines) + "\n"
+
+
+_TOML_ESCAPES = {
+    "\\": "\\\\",
+    '"': '\\"',
+    "\b": "\\b",
+    "\t": "\\t",
+    "\n": "\\n",
+    "\f": "\\f",
+    "\r": "\\r",
+}
+
+
+def _toml(value: str) -> str:
+    """Render a TOML basic string, escaping quotes and control characters."""
+    chars: list[str] = []
+    for ch in value:
+        if ch in _TOML_ESCAPES:
+            chars.append(_TOML_ESCAPES[ch])
+        elif ord(ch) < 0x20 or ord(ch) == 0x7F:
+            chars.append(f"\\u{ord(ch):04X}")
+        else:
+            chars.append(ch)
+    return '"' + "".join(chars) + '"'

src/weekly_activity/wizard.py

@@ -0,0 +1,398 @@
+"""Interactive onboarding wizard behind `weekly-activity config`.
+
+A small menu-driven editor for the account-list config: it shows the
+current sources (secrets masked), then offers Add a new source /
+Remove a source / Exit, supporting several accounts per provider.
+
+Adding a non-anonymous Launchpad account runs launchpadlib's interactive
+OAuth flow right here (the browser opens once and the token lands in the
+system keyring) so `weekly-activity report` never has to authenticate;
+supplying a credentials file skips that flow. Token values are never
+echoed back.
+"""
+
+from __future__ import annotations
+
+import getpass
+import os
+import sys
+from collections.abc import Callable, Sequence
+from pathlib import Path
+from typing import TYPE_CHECKING, cast
+
+from weekly_activity.aggregate import source_label
+from weekly_activity.config import (
+    ActivityConfig,
+    BtsSettings,
+    ConfigError,
+    GithubSettings,
+    GitlabSettings,
+    LaunchpadSettings,
+    load_config,
+    save_config,
+)
+from weekly_activity.sources.launchpad import LaunchpadSource
+
+if TYPE_CHECKING:
+    from prompt_toolkit.key_binding import KeyBindings, KeyPressEvent
+
+
+_APPLICATION_NAME = "weekly-activity"
+
+# Stable processing order, shared by menus and summaries: (config attr, menu text).
+_KINDS: tuple[tuple[str, str], ...] = (
+    ("launchpad", "Launchpad account"),
+    ("github", "GitHub account"),
+    ("gitlab", "GitLab instance/account"),
+    ("bts", "Debian BTS email"),
+)
+
+
+def run_wizard(config_path: Path) -> None:
+    """Edit the source configuration interactively; write when changed."""
+    try:
+        config = _load_existing(config_path)
+        original = _snapshot(config)
+        print()
+        print("Welcome! This wizard manages the sources aggregated by")
+        print("`weekly-activity report`. Enter a menu number, accept [defaults]")
+        print("with Enter, and pick Exit to save; Ctrl-C aborts without writing.")
+        try:
+            _menu_loop(config)
+        except EOFError:
+            print()
+            print("End of input; applying current selections.")
+    except KeyboardInterrupt:
+        raise SystemExit(130) from None  # parity with secret-prompt interruption
+    _finish(config, original, config_path)
+
+
+def _snapshot(config: ActivityConfig) -> ActivityConfig:
+    """Copy the lists so edits can be compared against the starting point."""
+    return ActivityConfig(
+        launchpad=list(config.launchpad),
+        github=list(config.github),
+        gitlab=list(config.gitlab),
+        bts=list(config.bts),
+    )
+
+
+def _menu_loop(config: ActivityConfig) -> None:
+    while True:
+        print()
+        print("Current sources:")
+        print(_describe(config))
+        action = _menu("What do you want to do?", ["Add a new source", "Remove a source", "Exit"])
+        if action == 0:
+            _add_source_flow(config)
+        elif action == 1:
+            _remove_source_flow(config)
+        else:
+            if not config.enabled_sources() and not _confirm(
+                "No sources are configured; write an empty configuration anyway?", default=False
+            ):
+                continue
+            return
+
+
+def _finish(config: ActivityConfig, original: ActivityConfig, config_path: Path) -> None:
+    if config == original:
+        print()
+        print("No changes made; configuration left untouched.")
+        return
+    save_config(config, config_path)
+    enabled = ", ".join(config.enabled_sources())
+    print()
+    print(f"Configuration written to {config_path}.")
+    print(f"Enabled sources: {enabled}.")
+    print(_describe(config, indent=""))
+    print("Tokens are never printed back. Run `weekly-activity report` to")
+    print("aggregate activity across all of them in one output.")
+
+
+def _load_existing(path: Path) -> ActivityConfig:
+    try:
+        return load_config(path)
+    except FileNotFoundError:
+        return ActivityConfig()
+    except ConfigError as exc:
+        print(f"warning: ignoring unusable existing config: {exc}", file=sys.stderr)
+        print(f"warning: {path} will be replaced by a fresh configuration.", file=sys.stderr)
+        return ActivityConfig()
+
+
+# --- Add / remove flows ---
+
+
+def _add_source_flow(config: ActivityConfig) -> None:
+    chosen = _menu(
+        "Which source do you want to add?", [text for _, text in _KINDS], cancellable=True
+    )
+    if chosen is None:
+        return
+    kind = _KINDS[chosen][0]
+    account = _ADDERS[kind]()
+    if account is None:
+        print("Cancelled; nothing was added.")
+        return
+    accounts = getattr(config, kind)
+    accounts.append(account)
+    label = source_label(kind, _display_name(account), _identity(account), len(accounts) > 1)
+    print(f"Added: {label} ({_identity(account)}).")
+
+
+def _remove_source_flow(config: ActivityConfig) -> None:
+    entries: list[tuple[str, int]] = []
+    options: list[str] = []
+    for kind, _ in _KINDS:
+        accounts = getattr(config, kind)
+        for index, account in enumerate(accounts):
+            entries.append((kind, index))
+            options.append(_account_line(kind, account, len(accounts) > 1))
+    if not entries:
+        print("There are no sources to remove yet.")
+        return
+    picked = _menu("Which account do you want to remove?", options, cancellable=True)
+    if picked is None or not _confirm(f"Remove {options[picked]}?", default=False):
+        return
+    kind, index = entries[picked]
+    del getattr(config, kind)[index]
+    print(f"Removed: {options[picked]}.")
+
+
+def _add_github() -> GithubSettings | None:
+    print("GitHub identity is the profile @handle.")
+    username = _ask_required("GitHub username")
+    name = _ask("Display name (optional label)")
+    print("Leave both token fields empty to fall back to $GITHUB_TOKEN or `gh auth token`.")
+    token_env = _ask("Env var holding the token (optional, e.g. GITHUB_TOKEN)")
+    token = _ask_secret("GitHub token (input hidden)")
+    return GithubSettings(username=username, token=token, token_env=token_env, name=name)
+
+
+def _add_gitlab() -> GitlabSettings | None:
+    print("GitLab needs the instance URL plus your username there.")
+    username = _ask_required("GitLab username")
+    url = _ask("Instance URL", "https://gitlab.com")
+    name = _ask("Display name (optional label)")
+    print("Leave both token fields empty to discover the token from the environment.")
+    token = _ask_secret("API token (input hidden)")
+    token_env = _ask("Env var holding the token (optional, e.g. SALSA_TOKEN)")
+    return GitlabSettings(username=username, url=url, token=token, token_env=token_env, name=name)
+
+
+def _add_bts() -> BtsSettings | None:
+    print("The BTS identifies people by the email used on bugs (submitter/owner).")
+    email = _ask_required("Email address on Debian bugs")
+    name = _ask("Display name (optional label)")
+    return BtsSettings(email=email, name=name)
+
+
+def _add_launchpad() -> LaunchpadSettings | None:
+    print("Launchpad identity is the ~name shown on your profile page.")
+    username = _ask_required("Launchpad username")
+    name = _ask("Display name (optional label)")
+    anonymous = _confirm("Anonymous mode (public data only, no OAuth)?", default=True)
+    if anonymous:
+        return LaunchpadSettings(username=username, mode="anonymous", name=name)
+
+    credentials_file = _ask("Credentials file path (empty to authenticate right now)")
+    if credentials_file:
+        expanded = os.path.expanduser(credentials_file)
+        if not os.path.isfile(expanded):
+            print(f"note: {expanded} does not exist yet; it will be read at report time.")
+        print(f"OAuth credentials will be read from {credentials_file} at report time.")
+        return LaunchpadSettings(
+            username=username, mode="credentials", credentials_file=credentials_file, name=name
+        )
+
+    # Real config-time OAuth: construct LaunchpadSource (this runs
+    # launchpadlib's browser flow) and keep only its side effect — the token
+    # in the system keyring keyed to the application name.
+    while True:
+        print("A browser window will open to authorize weekly-activity with Launchpad.")
+        try:
+            LaunchpadSource(
+                service="production",
+                anonymous=False,
+                application_name=_APPLICATION_NAME,
+                credentials_file=None,
+            )
+        except Exception as exc:  # noqa: BLE001 - network/user errors here are recoverable
+            print(f"warning: Launchpad authentication failed: {exc}")
+            if not _confirm("Retry authentication?", default=False):
+                return None
+        else:
+            break
+    print("Launchpad authorization complete: the token is stored in your system keyring;")
+    print("`weekly-activity report` will reuse it automatically.")
+    return LaunchpadSettings(username=username, mode="credentials", name=name)
+
+
+_ADDERS: dict[str, Callable[[], object | None]] = {
+    "launchpad": _add_launchpad,
+    "github": _add_github,
+    "gitlab": _add_gitlab,
+    "bts": _add_bts,
+}
+
+
+# --- Display helpers (secrets masked) ---
+
+
+def _describe(config: ActivityConfig, indent: str = "  ") -> str:
+    """Human-readable summary masking every token value."""
+    lines: list[str] = []
+    for kind, _ in _KINDS:
+        accounts = getattr(config, kind)
+        for account in accounts:
+            lines.append(indent + _account_line(kind, account, len(accounts) > 1))
+    return "\n".join(lines) if lines else f"{indent}(no sources configured)"
+
+
+def _account_line(kind: str, account: object, ambiguous: bool) -> str:
+    identity = _identity(account)
+    prefix = f"{source_label(kind, _display_name(account), identity, ambiguous)}: "
+    if kind == "launchpad" and isinstance(account, LaunchpadSettings):
+        auth = (
+            "anonymous (public data only)"
+            if account.mode == "anonymous"
+            else f"OAuth via {account.credentials_file or 'system keyring'}"
+        )
+        return f"{prefix}{identity} — {auth}"
+    if kind == "github" and isinstance(account, GithubSettings):
+        fallback = account.token_env or "GITHUB_TOKEN / gh"
+        token_desc = "stored" if account.token else f"not stored (uses {fallback})"
+        return f"{prefix}{identity} — token {token_desc}"
+    if kind == "gitlab" and isinstance(account, GitlabSettings):
+        fallback = account.token_env or "<HOST>_TOKEN / GITLAB_TOKEN env"
+        token_desc = "stored" if account.token else f"not stored ({fallback})"
+        return f"{prefix}{identity} @ {account.url} — token {token_desc}"
+    if kind == "bts" and isinstance(account, BtsSettings):
+        return f"{prefix}{identity}"
+    return f"{prefix}{identity}"  # pragma: no cover - exhaustive over known kinds
+
+
+def _identity(account: object) -> str:
+    if isinstance(account, BtsSettings):
+        return account.email
+    if isinstance(account, (GithubSettings, GitlabSettings, LaunchpadSettings)):
+        return account.username
+    return ""  # pragma: no cover - exhaustive over known kinds
+
+
+def _display_name(account: object) -> str:
+    if isinstance(account, (GithubSettings, GitlabSettings, LaunchpadSettings, BtsSettings)):
+        return account.name
+    return ""  # pragma: no cover - exhaustive over known kinds
+
+
+# --- Prompt helpers ---
+
+
+def _menu(title: str, options: Sequence[str], *, cancellable: bool = False) -> int | None:
+    """Pick an option: interactive arrows on a TTY, numbered prompt otherwise.
+
+    Returns the selected index, or None when backed out of a cancellable menu.
+    """
+    if sys.stdin.isatty() and sys.stdout.isatty():
+        return _tty_menu(title, options, cancellable=cancellable)
+    return _numbered_menu(title, options, cancellable=cancellable)
+
+
+def _numbered_menu(title: str, options: Sequence[str], *, cancellable: bool = False) -> int | None:
+    """Numbered-choice fallback for non-TTY stdin (piped input, tests, CI)."""
+    print(title)
+    for number, option in enumerate(options, 1):
+        print(f"  {number}) {option}")
+    back_number = len(options) + 1 if cancellable else 0
+    if cancellable:
+        print(f"  {back_number}) Back")
+    while True:
+        answer = input("Enter choice: ").strip()
+        if not answer and cancellable:
+            return None
+        if answer.isdigit():
+            number = int(answer)
+            if 1 <= number <= len(options):
+                return number - 1
+            if cancellable and number == back_number:
+                return None
+        valid = back_number or len(options)
+        print(f"Please enter a number between 1 and {valid}.")
+
+
+# --- Arrow-key menu (interactive TTYs only) ---
+
+
+# Sentinel handed back through the prompt when Esc backs out; it can never
+# collide with a real choice because every choice's value is its list index.
+_CANCELLED = object()
+
+
+def _tty_menu(title: str, options: Sequence[str], *, cancellable: bool = False) -> int | None:
+    """Arrow-key menu rendered by questionary instead of hand-rolled ANSI.
+
+    Up/Down (plus j/k) move the bold ``>``-marked highlight and Enter
+    accepts; Esc or Ctrl-C backs out of a cancellable menu while Ctrl-C
+    alone aborts fixed ones. questionary is imported lazily so
+    non-interactive commands never pay for prompt-toolkit.
+    """
+    import questionary
+
+    choices = [questionary.Choice(title=text, value=index) for index, text in enumerate(options)]
+    style = questionary.Style([("pointer", "bold"), ("selected", "bold")])
+    question = questionary.select(title, choices, pointer=">", style=style)
+    if cancellable:
+        # questionary wires arrows, Enter and Ctrl-C itself but leaves Esc as
+        # a no-op; graft back-out onto this prompt's own key-binding registry.
+        registry = cast("KeyBindings", question.application.key_bindings)
+
+        @registry.add("escape", eager=True)
+        def _back_out(event: KeyPressEvent) -> None:
+            event.app.exit(result=_CANCELLED)
+
+    try:
+        picked = question.unsafe_ask()
+    except (KeyboardInterrupt, EOFError):
+        if not cancellable:
+            raise  # Ctrl-C aborts fixed menus, like every other wizard input
+        return None
+    if not isinstance(picked, int):  # the Esc sentinel, or any surprise value
+        return None
+    return picked
+
+
+def _ask(prompt: str, default: str = "") -> str:
+    suffix = f" [{default}]" if default else ""
+    answer = input(f"{prompt}{suffix}: ").strip()
+    return answer or default
+
+
+def _ask_required(prompt: str, default: str = "") -> str:
+    while True:
+        answer = _ask(prompt, default)
+        if answer:
+            return answer
+        print("A value is required here.")
+
+
+def _ask_secret(prompt: str, default: str = "") -> str:
+    """Read a secret without echo; keep the stored one when input is empty."""
+    try:
+        entered = getpass.getpass(f"{prompt}: ")
+    except (EOFError, KeyboardInterrupt):
+        raise SystemExit(130) from None
+    return entered.strip() or default
+
+
+def _confirm(prompt: str, *, default: bool = False) -> bool:
+    hint = "[Y/n]" if default else "[y/N]"
+    while True:
+        answer = input(f"{prompt} {hint}: ").strip().lower()
+        if not answer:
+            return default
+        if answer in {"y", "yes"}:
+            return True
+        if answer in {"n", "no"}:
+            return False

tests/test_aggregate.py

@@ -0,0 +1,172 @@
+"""Unit tests for weekly_activity.aggregate — stubbed sources, no network."""
+
+from __future__ import annotations
+
+import os
+import unittest
+from datetime import datetime
+from unittest import mock
+
+from weekly_activity.aggregate import (
+    SourceSpec,
+    build_specs,
+    collect_specs,
+    format_combined,
+)
+from weekly_activity.config import (
+    ActivityConfig,
+    BtsSettings,
+    GithubSettings,
+    GitlabSettings,
+    LaunchpadSettings,
+)
+from weekly_activity.model import ActivityReport, Section
+
+
+class StubSource:
+    def __init__(self, name: str = "stub", *, fail: bool = False) -> None:
+        self.name = name
+        self._fail = fail
+
+    def collect(self, username: str, since: datetime, until: datetime) -> ActivityReport:
+        if self._fail:
+            raise RuntimeError("kaput")
+        return ActivityReport(
+            source=self.name,
+            username=username,
+            since=since,
+            until=until,
+            sections=[Section("Things", ["did a thing"])],
+            warnings=["some query failed"],
+        )
+
+
+def window() -> tuple[datetime, datetime]:
+    return datetime(2026, 8, 19), datetime(2026, 8, 26)
+
+
+class BuildSpecsTest(unittest.TestCase):
+    def test_maps_every_section_in_order(self) -> None:
+        config = ActivityConfig(
+            launchpad=[LaunchpadSettings(username="lp")],
+            github=[GithubSettings(username="gh")],
+            gitlab=[GitlabSettings(username="gl")],
+            bts=[BtsSettings(email="bts@mail")],
+        )
+        specs = build_specs(config)
+        self.assertEqual([s.name for s in specs], ["launchpad", "github", "gitlab", "bts"])
+        self.assertEqual([s.username for s in specs], ["lp", "gh", "gl", "bts@mail"])
+
+    def test_multiple_accounts_per_kind_in_listing_order(self) -> None:
+        config = ActivityConfig(
+            launchpad=[
+                LaunchpadSettings(username="lp-a"),
+                LaunchpadSettings(username="lp-b", mode="credentials"),
+            ],
+            github=[
+                GithubSettings(username="gh-a", name="work"),
+                GithubSettings(username="gh-b"),
+            ],
+            bts=[
+                BtsSettings(email="one@x.org"),
+                BtsSettings(email="two@x.org", name="two"),
+            ],
+        )
+        specs = build_specs(config)
+        # All launchpad accounts first, then github, then bts; listing order kept.
+        self.assertEqual(
+            [s.username for s in specs], ["lp-a", "lp-b", "gh-a", "gh-b", "one@x.org", "two@x.org"]
+        )
+        self.assertEqual(
+            [s.name for s in specs],
+            [
+                "launchpad/lp-a",  # siblings force identity into the label
+                "launchpad/lp-b",
+                "github/work",  # explicit label wins even among siblings
+                "github/gh-b",
+                "bts/one@x.org",
+                "bts/two",
+            ],
+        )
+
+    def test_single_unnamed_account_keeps_plain_kind_label(self) -> None:
+        config = ActivityConfig(github=[GithubSettings(username="only-one")])
+        specs = build_specs(config)
+        self.assertEqual(specs[0].name, "github")
+
+    def test_disabled_sections_are_skipped(self) -> None:
+        config = ActivityConfig(bts=[BtsSettings(email="only@one")])
+        self.assertEqual([s.name for s in build_specs(config)], ["bts"])
+
+    def test_launchpad_spec_expands_tilde_credentials_file(self) -> None:
+        config = ActivityConfig(launchpad=[LaunchpadSettings(username="anon")])
+        config.launchpad.append(
+            LaunchpadSettings(mode="credentials", username="u", credentials_file="~/creds.json")
+        )
+        specs = build_specs(config)
+        with mock.patch("weekly_activity.aggregate.LaunchpadSource") as fake_source:
+            for spec in specs:
+                spec.make()
+        anon_kwargs, cred_kwargs = (call.kwargs for call in fake_source.call_args_list)
+        self.assertTrue(anon_kwargs["anonymous"])
+        self.assertIsNone(anon_kwargs["credentials_file"])
+        self.assertFalse(cred_kwargs["anonymous"])
+        self.assertEqual(cred_kwargs["credentials_file"], os.path.expanduser("~") + "/creds.json")
+        # The two accounts must not share one late-bound factory.
+        self.assertNotEqual(anon_kwargs["credentials_file"], cred_kwargs["credentials_file"])
+
+    def test_github_token_env_resolved_lazily(self) -> None:
+        config = ActivityConfig(github=[GithubSettings(username="u", token_env="MY_GH_TOKEN")])
+        spec = build_specs(config)[0]
+        with (
+            mock.patch("weekly_activity.aggregate.GitHubSource") as fake_source,
+            mock.patch.dict(os.environ, {"MY_GH_TOKEN": "env-token"}),
+        ):
+            spec.make()
+        self.assertEqual(fake_source.call_args.kwargs["token"], "env-token")
+
+    def test_github_stored_token_takes_precedence_over_env(self) -> None:
+        config = ActivityConfig(github=[GithubSettings(username="u", token="tok")])
+        spec = build_specs(config)[0]
+        with (
+            mock.patch("weekly_activity.aggregate.GitHubSource") as fake_source,
+            mock.patch.dict(os.environ, {"MY_GH_TOKEN": "env-token"}),
+        ):
+            spec.make()
+        self.assertEqual(fake_source.call_args.kwargs["token"], "tok")
+
+
+class CollectAndFormatTest(unittest.TestCase):
+    def test_failure_isolated_per_source(self) -> None:
+        since, until = window()
+        specs = [
+            SourceSpec(name="alpha", username="a", make=lambda: StubSource("alpha")),
+            SourceSpec(name="beta", username="b", make=lambda: StubSource("beta", fail=True)),
+        ]
+        collected = collect_specs(specs, since, until)
+        self.assertIsNotNone(collected[0].report)
+        self.assertIn("RuntimeError: kaput", collected[1].error or "")
+
+        text = format_combined(collected, since, until)
+        self.assertIn("Weekly activity report — 2 source(s)", text)
+        self.assertIn("Activity report — alpha / ~a", text)
+        self.assertIn("[skipped: beta — RuntimeError: kaput]", text)
+
+    def test_heading_uses_account_label_not_bare_source_name(self) -> None:
+        since, until = window()
+        specs = [
+            SourceSpec(name="github/work", username="acme-jane", make=lambda: StubSource("github"))
+        ]
+        text = format_combined(collect_specs(specs, since, until), since, until)
+        self.assertIn("Activity report — github/work / ~acme-jane", text)
+
+    def test_warning_surfaced_inside_block(self) -> None:
+        since, until = window()
+        specs = [SourceSpec(name="gamma", username="g", make=lambda: StubSource("gamma"))]
+        text = format_combined(collect_specs(specs, since, until), since, until)
+        self.assertIn("Warnings (partial data — some queries failed):", text)
+        self.assertIn("some query failed", text)
+
+
+if __name__ == "__main__":
+    unittest.main()

tests/test_cli.py

@@ -0,0 +1,76 @@
+"""Unit tests for the report command's loading-status presentation layer."""
+
+from __future__ import annotations
+
+import io
+import sys
+import unittest
+from datetime import datetime
+from unittest import mock
+
+from weekly_activity.aggregate import SourceSpec, collect_specs
+from weekly_activity.cli import _collect_report_sources
+from weekly_activity.model import ActivityReport, Section
+
+
+class StubSource:
+    """Deterministic stand-in satisfying the ActivitySource interface."""
+
+    def __init__(self, *, fail: bool = False) -> None:
+        self._fail = fail
+        self.name = "stub"
+
+    def collect(self, username: str, since: datetime, until: datetime) -> ActivityReport:
+        if self._fail:
+            raise RuntimeError("some query failed")
+        return ActivityReport(
+            source=username,
+            username=username,
+            since=since,
+            until=until,
+            sections=[Section(title="Patches", entries=["one commit"])],
+        )
+
+
+def spec(name: str, *, fail: bool = False) -> SourceSpec:
+    source = StubSource(fail=fail)
+    return SourceSpec(name=name, username="me", make=lambda: source)
+
+
+WINDOW = (datetime(2026, 8, 19), datetime(2026, 8, 26))
+
+
+class FakeTty(io.StringIO):
+    """Capture buffer that claims to be a terminal so status lines activate."""
+
+    def isatty(self) -> bool:
+        return True
+
+
+class ReportStatusTest(unittest.TestCase):
+    def test_status_is_silent_without_a_tty(self) -> None:
+        specs = [spec("alpha"), spec("beta")]
+        buffer = io.StringIO()
+        with mock.patch.object(sys, "stdout", buffer):
+            collected = _collect_report_sources(specs, *WINDOW)
+        self.assertEqual(buffer.getvalue(), "")
+        self.assertEqual(collected, collect_specs(specs, *WINDOW))
+
+    def test_status_names_current_source_then_clears_on_a_tty(self) -> None:
+        specs = [spec("launchpad"), spec("github/personal"), spec("broken", fail=True)]
+        buffer = FakeTty()
+        with mock.patch.object(sys, "stdout", buffer):
+            collected = _collect_report_sources(specs, *WINDOW)
+        text = buffer.getvalue()
+        self.assertIn("# Generating report: pulling launchpad data...", text)
+        self.assertIn("# Generating report: pulling github/personal data...", text)
+        self.assertNotIn("# Generating report: pulling broken data...\n", text)
+        # The final erase leaves nothing visible behind the transient status.
+        self.assertTrue(text.endswith("\x1b[2K\r"))
+        self.assertEqual(collected, collect_specs(specs, *WINDOW))
+        errors = [item.error for item in collected if item.error is not None]
+        self.assertTrue(errors and all("RuntimeError" in err for err in errors))
+
+
+if __name__ == "__main__":
+    unittest.main()

tests/test_config.py

@@ -0,0 +1,258 @@
+"""Unit tests for weekly_activity.config — pure logic, no network."""
+
+from __future__ import annotations
+
+import os
+import tempfile
+import unittest
+from pathlib import Path
+
+from weekly_activity.config import (
+    ActivityConfig,
+    BtsSettings,
+    ConfigError,
+    GithubSettings,
+    GitlabSettings,
+    LaunchpadSettings,
+    default_config_path,
+    format_toml,
+    load_config,
+    save_config,
+)
+
+
+def load_config_from_text(text: str) -> ActivityConfig:
+    """Helper: parse TOML through the public loader without touching disk."""
+    with tempfile.TemporaryDirectory() as tmp:
+        path = Path(tmp) / "weekly-activity.toml"
+        path.write_text(text, encoding="utf-8")
+        return load_config(path)
+
+
+def sample_v2_config() -> ActivityConfig:
+    return ActivityConfig(
+        launchpad=[
+            LaunchpadSettings(
+                username="lp-user",
+                mode="credentials",
+                credentials_file="/tmp/lp-creds.json",
+                service="staging",
+                name="work",
+            ),
+            LaunchpadSettings(username="anon-lp"),
+        ],
+        github=[
+            GithubSettings(username='gh"user', token="tok\\en"),
+            GithubSettings(username="acme-jane", token_env="WORK_GH_TOKEN", name="work"),
+        ],
+        gitlab=[
+            GitlabSettings(
+                username="gl-user",
+                url="https://salsa.debian.org",
+                token_env="SALSA_TOKEN",
+                name="salsa",
+            )
+        ],
+        bts=[BtsSettings(email="dev@example.org")],
+    )
+
+
+class ConfigRoundTripTest(unittest.TestCase):
+    def test_round_trip_preserves_all_accounts(self) -> None:
+        config = sample_v2_config()
+        with tempfile.TemporaryDirectory() as tmp:
+            path = Path(tmp) / "weekly-activity.toml"
+            save_config(config, path)
+            self.assertEqual(load_config(path), config)
+
+    def test_save_writes_v2_version_header(self) -> None:
+        text = format_toml(ActivityConfig(github=[GithubSettings(username="g")]))
+        self.assertIn("version = 2", text)
+        self.assertIn("[[github.accounts]]", text)
+
+    def test_save_creates_owner_only_file(self) -> None:
+        config = ActivityConfig(bts=[BtsSettings(email="x@example.org")])
+        with tempfile.TemporaryDirectory() as tmp:
+            path = Path(tmp) / "nested" / "cfg.toml"
+            save_config(config, path)
+            self.assertEqual(os.stat(path).st_mode & 0o777, 0o600)
+
+    def test_empty_config_emits_header_only(self) -> None:
+        text = format_toml(ActivityConfig())
+        self.assertEqual(load_config_from_text(text), ActivityConfig())
+        self.assertNotIn("[[", text)
+
+    def test_empty_optionals_round_trip_to_defaults(self) -> None:
+        config = ActivityConfig(
+            github=[GithubSettings(username="octocat")],
+            bts=[BtsSettings(email="a@b.c")],
+        )
+        reparsed = load_config_from_text(format_toml(config))
+        self.assertEqual(reparsed.github, [GithubSettings(username="octocat")])
+        self.assertEqual(reparsed.bts, [BtsSettings(email="a@b.c")])
+        self.assertEqual(reparsed.launchpad, [])
+
+    def test_escapes_quotes_backslashes_and_newlines(self) -> None:
+        config = ActivityConfig(github=[GithubSettings(username='a"b\\c\nd')])
+        reparsed = load_config_from_text(format_toml(config))
+        self.assertEqual(reparsed.github[0].username, 'a"b\\c\nd')
+
+    def test_load_does_not_rewrite_file(self) -> None:
+        with tempfile.TemporaryDirectory() as tmp:
+            path = Path(tmp) / "weekly-activity.toml"
+            original = "[github]\nusername = 'octo'\n"
+            path.write_text(original, encoding="utf-8")
+            before = path.read_bytes()
+            config = load_config(path)
+            self.assertEqual(config.github, [GithubSettings(username="octo")])
+            self.assertEqual(path.read_bytes(), before)
+
+
+class ConfigLoadingTest(unittest.TestCase):
+    def test_missing_file_raises_filenotfound(self) -> None:
+        with self.assertRaises(FileNotFoundError):
+            load_config(Path("/nonexistent/weekly-activity.toml"))
+
+    def test_unknown_section_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text('version = 2\n\n[[gitea.accounts]]\nusername = "x"\n')
+
+    def test_unknown_top_level_key_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text("version = 2\nsources = 3\n")
+
+    def test_unknown_account_key_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text('[github]\naccounts = [{username = "x", tokken = "t"}]\n')
+
+    def test_section_must_hold_only_accounts_table(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text('[version]\nwrong = "shape"\n')  # wrong type below
+        with self.assertRaises(ConfigError):
+            load_config_from_text(
+                'version = 2\n\n[github]\nusername = "legacy"\n'  # v2 without .accounts
+            )
+
+    def test_accounts_must_be_list_of_tables(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text('version = 2\n\n[github]\naccounts = "nope"\n')
+        with self.assertRaises(ConfigError):
+            load_config_from_text("version = 2\n\n[gitlab]\naccounts = [42]\n")
+
+    def test_non_string_value_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text("version = 2\n\n[[github.accounts]]\nusername = 42\n")
+
+    def test_bad_launchpad_mode_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text(
+                'version = 2\n\n[[launchpad.accounts]]\nusername = "u"\nmode = "oauth2"\n'
+            )
+
+    def test_bad_launchpad_service_rejected(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text(
+                'version = 2\n\n[[launchpad.accounts]]\nusername = "u"\nservice = "qa"\n'
+            )
+
+    def test_v2_section_without_accounts_table_rejected(self) -> None:
+        # A version-tagged file must use the [[kind.accounts]] shape.
+        with self.assertRaises(ConfigError):
+            load_config_from_text('version = 2\n\n[github]\nusername = "legacy"\n')
+
+    def test_account_errors_carry_position(self) -> None:
+        text = (
+            "version = 2\n"
+            '\n[[launchpad.accounts]]\nusername = "ok"\n'
+            '\n[[launchpad.accounts]]\nusername = "bad"\nservice = "oops"\n'
+        )
+        with tempfile.TemporaryDirectory() as tmp:
+            path = Path(tmp) / "weekly-activity.toml"
+            path.write_text(text, encoding="utf-8")
+            with self.assertRaisesRegex(ConfigError, r"launchpad\.accounts\[1\]"):
+                load_config(path)
+
+    def test_enabled_sources_stable_order(self) -> None:
+        config = ActivityConfig(
+            bts=[BtsSettings(email="e")],
+            github=[GithubSettings(username="g")],
+        )
+        self.assertEqual(config.enabled_sources(), ["github", "bts"])
+
+
+class ConfigMigrationTest(unittest.TestCase):
+    V1_TEXT = """\
+[launchpad]
+username = "lp-user"
+mode = "credentials"
+credentials_file = "~/creds.json"
+
+[github]
+username = "octo"
+token = "t0k"
+
+[gitlab]
+url = "https://salsa.debian.org"
+username = "gl"
+token_env = "SALSA_TOKEN"
+
+[bts]
+email = "dev@example.org"
+"""
+
+    def test_v1_file_migrates_to_single_unnamed_accounts_in_memory(self) -> None:
+        config = load_config_from_text(self.V1_TEXT)
+        self.assertEqual(config.enabled_sources(), ["launchpad", "github", "gitlab", "bts"])
+        self.assertEqual(
+            config.launchpad,
+            [
+                LaunchpadSettings(
+                    username="lp-user",
+                    mode="credentials",
+                    credentials_file="~/creds.json",
+                )
+            ],
+        )
+        self.assertEqual(config.github, [GithubSettings(username="octo", token="t0k")])
+        self.assertEqual(
+            config.gitlab,
+            [
+                GitlabSettings(
+                    url="https://salsa.debian.org",
+                    username="gl",
+                    token_env="SALSA_TOKEN",
+                )
+            ],
+        )
+        self.assertEqual(config.bts, [BtsSettings(email="dev@example.org")])
+
+    def test_saving_migrated_config_persists_v2_shape(self) -> None:
+        config = load_config_from_text(self.V1_TEXT)
+        text = format_toml(config)
+        self.assertIn("version = 2", text)
+        self.assertEqual(load_config_from_text(text), config)
+
+    def test_v1_unknown_keys_still_strict(self) -> None:
+        with self.assertRaises(ConfigError):
+            load_config_from_text('[github]\nusername = "u"\ntokken = "t"\n')
+
+    def test_v2_shaped_file_without_version_rejected(self) -> None:
+        with self.assertRaisesRegex(ConfigError, "version"):
+            load_config_from_text('[[github.accounts]]\nusername = "u"\n')
+
+
+class ConfigPathTest(unittest.TestCase):
+    def test_default_path_uses_xdg_or_home(self) -> None:
+        old = os.environ.get("XDG_CONFIG_HOME")
+        try:
+            os.environ["XDG_CONFIG_HOME"] = "/custom/cfg"
+            self.assertEqual(default_config_path(), Path("/custom/cfg/weekly-activity.toml"))
+            del os.environ["XDG_CONFIG_HOME"]
+            self.assertTrue(str(default_config_path()).endswith("/.config/weekly-activity.toml"))
+        finally:
+            if old is not None:
+                os.environ["XDG_CONFIG_HOME"] = old
+
+
+if __name__ == "__main__":
+    unittest.main()

tests/test_wizard.py

@@ -0,0 +1,307 @@
+"""Unit tests for the wizard menus — scripted input, no raw terminal involved."""
+
+from __future__ import annotations
+
+import io
+import os
+import pty
+import re
+import select
+import subprocess
+import sys
+import time
+import types
+import unittest
+from pathlib import Path
+from typing import Any, cast
+from unittest import mock
+
+from weekly_activity.wizard import _CANCELLED, _menu
+
+
+class NumberedMenuFallbackTest(unittest.TestCase):
+    """When stdin/stdout are not a TTY the numbered prompt stays as it was."""
+
+    def run_numbered(self, answers: list[str], options: list[str], *, cancellable: bool = False):
+        """Run _menu with forced non-TTY stdio and scripted interactive answers."""
+        fake_stdin, fake_stdout = io.StringIO(), io.StringIO()
+        with (
+            mock.patch.object(sys, "stdin", fake_stdin),
+            mock.patch.object(sys, "stdout", fake_stdout),
+            mock.patch("builtins.input", side_effect=iter(answers)),
+        ):
+            picked = _menu("Pick one", options, cancellable=cancellable)
+        return picked, fake_stdout.getvalue()
+
+    def test_returns_zero_based_index_of_chosen_option(self) -> None:
+        picked, output = self.run_numbered(["2"], ["Alpha", "Beta", "Gamma"])
+        self.assertEqual(picked, 1)
+        self.assertIn("Pick one", output)
+        self.assertIn("  2) Beta", output)
+
+    def test_empty_answer_backs_out_of_cancellable_menu(self) -> None:
+        picked, _ = self.run_numbered([""], ["Alpha", "Beta"], cancellable=True)
+        self.assertIsNone(picked)
+
+    def test_back_number_returns_none(self) -> None:
+        picked, _ = self.run_numbered(["3"], ["Alpha", "Beta"], cancellable=True)
+        self.assertIsNone(picked)
+
+    def test_invalid_answers_loop_until_valid_one(self) -> None:
+        picked, output = self.run_numbered(["9", "nope", "3"], ["Alpha", "Beta", "Gamma"])
+        self.assertEqual(picked, 2)
+        self.assertEqual(output.count("Please enter a number between 1 and 3."), 2)
+
+
+# --- Interactive (TTY) path -----------------------------------------------
+
+
+class _Tty(io.StringIO):
+    """A stream claiming to be a terminal so _menu routes to questionary."""
+
+    def isatty(self) -> bool:
+        return True
+
+
+class _FakeKeyBindings:
+    """Registry mimicking prompt_toolkit's enough to capture added handlers."""
+
+    def __init__(self) -> None:
+        self.added: list[tuple[tuple[Any, ...], Any]] = []
+
+    def add(self, *keys: Any, eager: bool = False):
+        def register(handler):
+            self.added.append((keys, handler))
+            return handler
+
+        return register
+
+
+class _FakeApplication:
+    def __init__(self) -> None:
+        self.key_bindings = _FakeKeyBindings()
+        self.exit_calls: list[dict[str, Any]] = []
+
+    def exit(self, result: Any = None, exception: BaseException | None = None, style=None) -> None:
+        self.exit_calls.append({"result": result, "exception": exception})
+
+
+class _FakeQuestion:
+    def __init__(self, outcome: Any) -> None:
+        self.application = _FakeApplication()
+        self.outcome = outcome
+
+    def unsafe_ask(self) -> Any:
+        if isinstance(self.outcome, BaseException):
+            raise self.outcome
+        return self.outcome
+
+
+class _FakeQuestionary:
+    """Stand-in exposing exactly the surface wizard._tty_menu touches."""
+
+    def __init__(self, outcome: Any) -> None:
+        self.outcome = outcome
+        self.select_outcome = _FakeQuestion(outcome)
+        self.seen: dict[str, Any] = {}
+
+    def Choice(self, **kwargs: Any) -> Any:
+        return types.SimpleNamespace(**kwargs)
+
+    @staticmethod
+    def Style(styles: list[tuple[str, str]]) -> list[tuple[str, str]]:
+        return styles
+
+    def select(self, message: str, choices: list[Any], **kwargs: Any) -> _FakeQuestion:
+        self.seen.update(message=message, choices=list(choices), **kwargs)
+        return self.select_outcome
+
+
+class QuestionaryDelegationTest(unittest.TestCase):
+    """On terminals _menu must hand selection to questionary, not re-implement it."""
+
+    def ask_tty(
+        self,
+        options: list[str],
+        outcome: Any,
+        *,
+        cancellable: bool,
+    ) -> tuple[int | None, _FakeQuestion, dict[str, Any]]:
+        """Run the TTY branch against an injected fake questionary module."""
+        fake = _FakeQuestionary(outcome)
+
+        with (
+            mock.patch.object(sys, "stdin", _Tty()),
+            mock.patch.object(sys, "stdout", _Tty()),
+            mock.patch.dict(sys.modules, {"questionary": cast("Any", fake)}),
+        ):
+            picked = _menu("Pick one", options, cancellable=cancellable)
+        return picked, fake.select_outcome, fake.seen
+
+    def test_non_cancellable_menu_delegates_and_maps_choice_value_to_index(self) -> None:
+        picked, question, seen = self.ask_tty(
+            ["Alpha", "Beta", "Gamma"],
+            outcome=2,
+            cancellable=False,
+        )
+        self.assertEqual(picked, 2)
+        self.assertEqual(seen["message"], "Pick one")
+        self.assertEqual(seen["pointer"], ">")
+        self.assertEqual(
+            [(choice.title, choice.value) for choice in seen["choices"]],
+            [("Alpha", 0), ("Beta", 1), ("Gamma", 2)],
+        )
+        # A fixed menu grafts nothing onto questionary's own bindings.
+        self.assertEqual(question.application.key_bindings.added, [])
+
+    def test_cancellable_menu_keeps_fixed_menus_behaviour_on_ctrl_c(self) -> None:
+        with self.assertRaises(KeyboardInterrupt):
+            self.ask_tty(["Alpha", "Beta"], outcome=KeyboardInterrupt(), cancellable=False)
+
+    def test_cancellable_menu_maps_ctrl_c_to_none(self) -> None:
+        picked, _, _ = self.ask_tty(
+            ["Alpha", "Beta"],
+            outcome=KeyboardInterrupt(),
+            cancellable=True,
+        )
+        self.assertIsNone(picked)
+
+    def test_cancellable_menu_grafts_escape_onto_the_prompt_bindings(self) -> None:
+        picked, question, _ = self.ask_tty(
+            ["Alpha", "Beta"],
+            outcome=_CANCELLED,
+            cancellable=True,
+        )
+        self.assertIsNone(picked)  # a non-int answer means "backed out"
+        keys, handler = question.application.key_bindings.added[-1]
+        self.assertIn("escape", keys)
+        handler(types.SimpleNamespace(app=question.application))
+        self.assertIs(question.application.exit_calls[0]["result"], _CANCELLED)
+
+
+# --- Real-terminal proof over a pty ----------------------------------------
+
+
+_MENU_CHILD = """
+import os
+import sys
+
+sys.path.insert(0, os.environ["WIZARD_SRC"])
+from weekly_activity.wizard import _tty_menu  # noqa: E402
+
+picked = _tty_menu("Pick one", ["Alpha", "Beta", "Gamma"], cancellable=True)
+print(f"PICKED={picked}", flush=True)
+"""
+
+_PICKED_LINE = re.compile(rb"PICKED=(\d+|None)")
+
+
+class ArrowMenuPtyTest(unittest.TestCase):
+    """Drive the genuine questionary menu across a real pty with raw keypresses.
+
+    This is the regression case the reviewer hit: arrow presses must move the
+    highlight of a live menu, which scripted ``io.StringIO`` stdin cannot reach.
+    """
+
+    TIMEOUT_S = 30.0
+
+    @classmethod
+    def setUpClass(cls) -> None:
+        cls.child_env = {
+            **os.environ,
+            "WIZARD_SRC": str(Path(__file__).resolve().parents[1] / "src"),
+        }
+
+    def drive_menu(self, sends: list[bytes]) -> tuple[str, int]:
+        """Render the menu on a pty, send keystrokes, return screen + exit code."""
+        master, slave = pty.openpty()
+        child = subprocess.Popen(
+            [sys.executable, "-c", _MENU_CHILD],
+            stdin=slave,
+            stdout=slave,
+            stderr=subprocess.PIPE,
+            env=self.child_env,
+        )
+        os.close(slave)
+        screen = bytearray()
+
+        try:
+            self._await(screen, master, child, lambda: b"Gamma" in screen, "menu render")
+            for payload in sends:
+                os.write(master, payload)
+                time.sleep(0.2)  # let the vt100 parser see keystrokes separately
+            self._await(
+                screen,
+                master,
+                child,
+                lambda: _PICKED_LINE.search(bytes(screen)) is not None,
+                "final PICKED line",
+            )
+            child.wait(timeout=self.TIMEOUT_S)
+            err = child.stderr.read().decode("utf-8", "replace") if child.stderr else ""
+            self.assertEqual(child.returncode, 0, f"menu child failed: {err}")
+            return bytes(screen).decode("utf-8", "replace"), child.returncode
+        finally:
+            if child.poll() is None:
+                child.kill()
+                child.wait()
+            if child.stderr is not None:
+                child.stderr.close()
+            os.close(master)
+
+    def _await(
+        self,
+        screen: bytearray,
+        master: int,
+        child: subprocess.Popen[bytes],
+        condition,
+        stage: str,
+    ) -> None:
+        """Read the pty until ``condition`` holds on accumulated output."""
+        deadline = time.monotonic() + self.TIMEOUT_S
+        while not condition():
+            remaining = deadline - time.monotonic()
+            if remaining <= 0:
+                break
+            readable, _, _ = select.select([master], [], [], min(remaining, 0.2))
+            if readable:
+                try:
+                    chunk = os.read(master, 4096)
+                except OSError:  # slave closed: the child exited early
+                    break
+                if chunk:
+                    screen.extend(chunk)
+        else:
+            return
+        if child.poll() is None:
+            child.kill()
+            child.wait(timeout=5)
+        err = child.stderr.read().decode("utf-8", "replace") if child.stderr else ""
+        self.fail(f"{stage}: stalled; tail={bytes(screen)[-300:]!r} stderr={err!r}")
+
+    def _picked(self, screen: str) -> str:
+        found = _PICKED_LINE.search(screen.encode())
+        if found is None:
+            self.fail(f"no PICKED line in {screen[-300:]!r}")
+        return found.group(1).decode()
+
+    def test_down_arrow_moves_highlight_and_enter_accepts(self) -> None:
+        # Start on "Alpha"; one Down must land on "Beta" — the reported bug.
+        screen, _ = self.drive_menu([b"\x1b[B", b"\r"])
+        self.assertEqual(self._picked(screen), "1")
+
+    def test_repeated_down_arrows_reach_third_option(self) -> None:
+        screen, _ = self.drive_menu([b"\x1b[B", b"\x1b[B", b"\r"])
+        self.assertEqual(self._picked(screen), "2")
+
+    def test_enter_without_movement_accepts_first_option(self) -> None:
+        screen, _ = self.drive_menu([b"\r"])
+        self.assertEqual(self._picked(screen), "0")
+
+    def test_escape_backs_out_of_a_cancellable_menu(self) -> None:
+        screen, _ = self.drive_menu([b"\x1b"])
+        self.assertEqual(self._picked(screen), "None")
+
+
+if __name__ == "__main__":
+    unittest.main()

uv.lock

@@ -381,6 +381,18 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/54/16/12b82f791c7f50ddec566873d5bdd245baa1491bac11d15ffb98aecc8f8b/pefile-2024.8.26-py3-none-any.whl", hash = "sha256:76f8b485dcd3b1bb8166f1128d395fa3d87af26360c2358fb75b80019b957c6f", size = 74766, upload-time = "2024-08-26T21:01:02.632Z" },
 ]
 
+[[package]]
+name = "prompt-toolkit"
+version = "3.0.53"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "wcwidth" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/7d/ea/39b988c938f75cb75d7045b5c69f8bfed47ee2152c8837fb403de29d6fb8/prompt_toolkit-3.0.53.tar.gz", hash = "sha256:9ec8a0ad96d5c56148b3f914aa79c1564c3fde5d2e6b876e7bc327e353cf8fa6", size = 435492, upload-time = "2026-07-26T20:56:14.758Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/54/6f/84908cad2d6aa5144abcf7b42709fe4fdb459bc640ec7ac5786e7693dabc/prompt_toolkit-3.0.53-py3-none-any.whl", hash = "sha256:01c0891d7f9237d5e339f7d3e42cdae80b7534abb1c7c0e3352efba6231492f2", size = 392288, upload-time = "2026-07-26T20:56:12.512Z" },
+]
+
 [[package]]
 name = "pycparser"
 version = "3.0"
@@ -457,6 +469,18 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/de/3d/8161f7711c017e01ac9f008dfddd9410dff3674334c233bde66e7ba65bbf/pywin32_ctypes-0.2.3-py3-none-any.whl", hash = "sha256:8a1513379d709975552d202d942d9837758905c8d01eb82b8bcc30918929e7b8", size = 30756, upload-time = "2024-08-14T10:15:33.187Z" },
 ]
 
+[[package]]
+name = "questionary"
+version = "2.1.1"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "prompt-toolkit" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/f6/45/eafb0bba0f9988f6a2520f9ca2df2c82ddfa8d67c95d6625452e97b204a5/questionary-2.1.1.tar.gz", hash = "sha256:3d7e980292bb0107abaa79c68dd3eee3c561b83a0f89ae482860b181c8bd412d", size = 25845, upload-time = "2025-08-28T19:00:20.851Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/3c/26/1062c7ec1b053db9e499b4d2d5bc231743201b74051c973dadeac80a8f43/questionary-2.1.1-py3-none-any.whl", hash = "sha256:a51af13f345f1cdea62347589fbb6df3b290306ab8930713bfae4d475a7d4a59", size = 36753, upload-time = "2025-08-28T19:00:19.56Z" },
+]
+
 [[package]]
 name = "ruff"
 version = "0.16.4"
@@ -550,6 +574,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/ec/f3/00b61e720165f73c6ef3c7a6ea732314c8ca1ba95b9ed1545fd788d9af9b/wadllib-2.1.0-py3-none-any.whl", hash = "sha256:41b58db0bb5fb21e188c7452281ee7b364b2690a4f6c388afea0ceaeafed1132", size = 61997, upload-time = "2026-07-01T10:48:01.37Z" },
 ]
 
+[[package]]
+name = "wcwidth"
+version = "0.8.2"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/34/74/c6428f875774288bec1396f5bfcbc2d925700a4dad61727fd5f2b12f249d/wcwidth-0.8.2.tar.gz", hash = "sha256:91fbef97204b96a3d4d421609b80340b760cf33e26da123ff243d76b1fda8dda", size = 1466253, upload-time = "2026-06-29T18:11:11.601Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/96/42/3e5985a0a7e57de470b320c6d6a1a67c844f6737a587f3d44dd13d1819e7/wcwidth-0.8.2-py3-none-any.whl", hash = "sha256:d63947694a0539a1d51e01eda7caf800c291020e6cdd7e28ad7b14dd33ad4f85", size = 323166, upload-time = "2026-06-29T18:11:09.888Z" },
+]
+
 [[package]]
 name = "weekly-activity"
 version = "0.3.0"
@@ -559,6 +592,7 @@ dependencies = [
     { name = "keyring" },
     { name = "launchpadlib" },
     { name = "python-debianbts" },
+    { name = "questionary" },
 ]
 
 [package.dev-dependencies]
@@ -576,6 +610,7 @@ requires-dist = [
     { name = "keyring", specifier = ">=25" },
     { name = "launchpadlib", specifier = ">=2.1.0" },
     { name = "python-debianbts", specifier = ">=4.1.1" },
+    { name = "questionary", specifier = ">=2.1.1" },
 ]
 
 [package.metadata.requires-dev]