Skip to content

Expose people reassign #5

Description

@ergofobe

The server exposes POST /api/v1/people/reassign and the Rust SDK has people::reassign(face_ids: &[String], person_id: Option<&str>) -> Result<()> (imogen-sdk/rust/src/people.rs:91), but the CLI's PeopleCommand (list / show / name / hide / merge / faces / status / enable) has no way to move a misgrouped face to another person — or to "unassign" it (person_id: None).

Sketch of the CLI shape, alongside the existing imogen people merge:

imogen people reassign <face-id>... --to <person>   # move faces to a person, by id or name
imogen people reassign <face-id>... --unassign       # person_id: None

--to <person> should resolve through the same find_person name-or-id lookup imogen people merge/imogen people show already use. imogen people faces <asset> (PeopleCommand::Faces) already prints face ids, so the natural flow is imogen people faces <asset> --ids | xargs imogen people reassign --to "Correct Name".

Not implementing this issue — filed as a tracking issue per an audit of imogen-cli's selection semantics (see PR against main from that audit).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions