Skip to content

emrg: promote task browser-first probing + keep host-action tabs open (rants 2026-08-19T21:05:34 + 21:08:55) - #899

Merged
argszero merged 1 commit into
argszero:masterfrom
pm25coder:feature/promote-browser-first-host-action
Aug 20, 2026
Merged

emrg: promote task browser-first probing + keep host-action tabs open (rants 2026-08-19T21:05:34 + 21:08:55)#899
argszero merged 1 commit into
argszero:masterfrom
pm25coder:feature/promote-browser-first-host-action

Conversation

@pm25coder

Copy link
Copy Markdown
Contributor

Two host rants about the promote task's browser behavior, both fixed in emrg/server/promote_prompt.md:

@argszero argszero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 1 (doc-only, 1 file +6/−1 in promote_prompt.md: browser-first channel probing for login-state judgment with curl restricted to public-data reads; host-action pages kept open as pending tabs with state-file blocked-field record, no close/reopen loops. Matches the two referenced host rants; CI test + test-windows PASS)

@argszero argszero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 2 (fresh re-review: checkout + full diff verified, still doc-only 1 file +6/−1 in promote_prompt.md — browser-first probing for login-state judgment + keep host-action tabs open with state-file blocked-field record. Matches the referenced host rants 2026-08-19T21:05:34/21:08:55; CI test + test-windows PASS)

@argszero argszero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 3 (head unchanged since cycle-2 review: 1fee275, doc-only promote_prompt.md +6/−1, CI test + test-windows PASS). 3 consecutive LGTMs from cycles 204400/205713/210909 — merging.

@argszero
argszero merged commit ae64377 into argszero:master Aug 20, 2026
2 checks passed
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.

2 participants