Skip to content

[13/36] Add OC-100 request type UX task - #47

Open
cjohnsto-nz wants to merge 1 commit into
supervisor/add-oc080-oc090-tasksfrom
supervisor/add-request-type-ux-task
Open

[13/36] Add OC-100 request type UX task#47
cjohnsto-nz wants to merge 1 commit into
supervisor/add-oc080-oc090-tasksfrom
supervisor/add-request-type-ux-task

Conversation

@cjohnsto-nz

@cjohnsto-nz cjohnsto-nz commented Jun 14, 2026

Copy link
Copy Markdown
Owner

@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-oc080-oc090-tasks branch from 7c7d3ba to 0038c19 Compare June 14, 2026 22:46
@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-request-type-ux-task branch from 2554e5d to 8bac182 Compare June 14, 2026 22:46
@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-oc080-oc090-tasks branch 2 times, most recently from a4f17ca to ba2aa43 Compare June 14, 2026 22:49
@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-request-type-ux-task branch from 8bac182 to def6356 Compare June 14, 2026 22:49
@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-oc080-oc090-tasks branch from ba2aa43 to 7b8fa8a Compare June 14, 2026 22:50
@cjohnsto-nz
cjohnsto-nz force-pushed the supervisor/add-request-type-ux-task branch from def6356 to 9246f62 Compare June 14, 2026 22:50
@cjohnsto-nz cjohnsto-nz changed the title Add request type UX task [13/21] Add OC-100 request type UX task Jun 14, 2026
@cjohnsto-nz cjohnsto-nz changed the title [13/21] Add OC-100 request type UX task [13/27] Add OC-100 request type UX task Jun 15, 2026
@cjohnsto-nz cjohnsto-nz changed the title [13/27] Add OC-100 request type UX task [13/36] Add OC-100 request type UX task Jun 15, 2026
@APKiwi

APKiwi commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Verdict: approve. Consistent numbering and cross-references, correct scope deferrals (execution to the protocol tracks, import/export to OC-070, auth to OC-050). One thing worth surfacing rather than leaving buried: the OC-090 supervisor report notes unassigned OC-050-looking edits (types.ts, oauth2Service.ts, oauth2TokenHelper.ts) sitting uncommitted alongside the OC-090 branch. That cross-track working-tree entanglement is recorded but never resolved, worth a follow-up so it doesn't surprise whoever rebases next.

@cjohnsto-nz

Copy link
Copy Markdown
Owner Author

Already resolved. The recorded working-tree entanglement was a transient parallel-agent state, not an outstanding stack issue.

The exact OC-050-looking edits in src/models/types.ts, src/services/oauth2Service.ts, and src/services/oauth2TokenHelper.ts were committed together in b271c9a (Complete OC-050 auth transport support) on PR #49, branch feature/oc-050-auth-transport. The currently verified PR #49 head is 4bcd234, and GitHub reports it MERGEABLE/CLEAN.

No change belongs on this OC-100 planning branch, and no follow-up branch is needed for those files. PR #47 and immediate child PR #48 are also MERGEABLE/CLEAN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants