Bring CLI documentation up to date

Complete the noun-first migration table and remove resolved TODOs. Also
describe cross-request partial failures for multi-option updates.

Author: myabc

CLAUDE.md

@@ -95,10 +95,11 @@ API response → `parser.Parse[SomethingDto]()` → `dto.Convert()` → `models.
 
 ## Testing conventions
 
-- Test files use external test packages: `package printer_test`, `package requests_test`
+- Component tests use external test packages (`package printer_test`, `package requests_test`, `package work_packages_test`); cmd-layer tests use internal packages (`package workpackage`) because they exercise unexported command handlers and flag variables
 - `TestMain` in `printer_test` initializes shared state (routes, printer) for the package
 - Tests use plain `t.Errorf` — no test framework, no assertions library
-- Tests only exist for `printer`, `requests`, `common`, and `configuration` — no tests on `cmd/` or `resources/`
+- Tests exist for `printer`, `requests`, `common`, `configuration`, `components/resources/work_packages`, and the `cmd/` packages (root, `activity`, `project`, `user`, `workpackage`)
+- Regression tests for command/resource behaviour use `httptest` localhost servers and count mutating requests to assert no-mutation guarantees
 - When adding a new printer function, add a corresponding test in `components/printer/`
 - When adding a new configuration function, add a corresponding test in `components/configuration/`
 

README.md

@@ -96,15 +96,21 @@ as `op NOUN VERB [additional information]`. You will see plenty of examples with
 Earlier releases (up to 0.5.5) used verb-first commands. These have been replaced without aliases;
 scripts need to migrate:
 
-| Old (verb-first)          | New (noun-first)          |
-|---------------------------|---------------------------|
-| `op list workpackages`    | `op work-package list`    |
-| `op create workpackage`   | `op work-package create`  |
-| `op update workpackage`   | `op work-package update`  |
-| `op inspect workpackage`  | `op work-package inspect` |
-| `op search workpackages`  | `op work-package search`  |
-| `op list projects`        | `op project list`         |
-| `op list notifications`   | `op notification list`    |
+| Old (verb-first)          | New (noun-first)                     |
+|---------------------------|--------------------------------------|
+| `op list workpackages`    | `op work-package list`               |
+| `op create workpackage`   | `op work-package create`             |
+| `op update workpackage`   | `op work-package update`             |
+| `op inspect workpackage`  | `op work-package inspect`            |
+| `op search workpackages`  | `op work-package search`             |
+| `op list projects`        | `op project list`                    |
+| `op inspect project 42`   | `op project inspect 42`              |
+| `op list notifications`   | `op notification list`               |
+| `op list activities 42`   | `op activity list --work-package 42` |
+| `op list status`          | `op status list`                     |
+| `op list types`           | `op type list`                       |
+| `op list timeentries`     | `op time-entry list`                 |
+| `op search user X`        | `op user search X`                   |
 
 ### Discoverability
 
@@ -272,9 +278,12 @@ op work-package update 42 --action Claim
 op work-package update PROJ-123 --action Claim
 
 # Batch updating some properties of a work package
-# Valid input will get processed, while invalid (e.g. wrongly typed) input will get omitted
 op work-package update 42 --subject 'The new subject' --type Implementation
 
+# Local inputs are validated before the first change is sent. Each API action
+# remains a separate request, so an earlier change may remain if a later
+# request fails.
+
 # Uploading an attachment to a work package
 op work-package update 42 --attach ./Downloads/Report.pdf
 ```

TODO.md

@@ -6,12 +6,6 @@ Issues identified during code review — non-blocking, to address in future iter
 
 - [ ] **`op time-entry create --activity` cannot list available activities** — `GET /api/v3/time_entries/activities` returns 404 in some OpenProject instances (version or permission issue). Investigate using the form endpoint `GET /api/v3/time_entries/form` which may include allowed activities in its schema.
 
-## Project identifier refactoring
-
-- [ ] **`validatedVersionId()` in `cmd/workpackage/list.go` swallows errors** — after `printer.Error(err)` the function continues with a nil `project`/`versions` instead of returning early. Should `return ""` immediately after the error print to avoid fragile control flow.
-
 ## CLI UX
 
-- [ ] **Add `-w` short flag to `activities --work-package`** — `--work-package` has no short option, inconsistent with similar flags elsewhere. Add `-w` or document why it's intentionally omitted.
-
 - [ ] **`--format` flag has no short option** — inconsistent with the convention of adding short flags for frequently used ones. Consider `-f` if it doesn't conflict.