Skip to content

fix: runCollection stability and putCollection noauth - #158

Closed
advancedresearcharray wants to merge 2 commits into
postmanlabs:mainfrom
advancedresearcharray:fix/runner-stability-put-noauth-107
Closed

fix: runCollection stability and putCollection noauth#158
advancedresearcharray wants to merge 2 commits into
postmanlabs:mainfrom
advancedresearcharray:fix/runner-stability-put-noauth-107

Conversation

@advancedresearcharray

@advancedresearcharray advancedresearcharray commented Jun 4, 2026 •
edited
Loading

Copy link
Copy Markdown

Summary

  • Harden stdio server: unhandledRejection / uncaughtException handlers
  • Newman: exception listener; wire stopOnError / abortOnError to bail
  • putCollection: add noauth to collection-level auth (parity with createCollection)

Addresses #107; partial #142 (full folder union schema still open)

Test plan

  • pnpm run build

root @cursoragent
fix: harden runCollection and add collection noauth on putCollection
8d63813 
- Log Newman exceptions; map stopOnError/abortOnError to bail options
- Register unhandledRejection/uncaughtException handlers on stdio server
- Add noauth to putCollection collection-level auth enum

Addresses #107 and part of #142

Co-authored-by: Cursor <cursoragent@cursor.com>
advancedresearcharray @cursoragent
refine: keep server alive on runCollection rejections and add unit tests
7d95647 
Log unhandled rejections after startup instead of exiting, wrap newman.run
in try/catch, and add unit coverage for bail mapping and putCollection noauth.

Co-authored-by: Cursor <cursoragent@cursor.com>
@advancedresearcharray
advancedresearcharray force-pushed the fix/runner-stability-put-noauth-107 branch from d05d746 to 7d95647 Compare June 8, 2026 03:46
@advancedresearcharray

advancedresearcharray commented Jun 12, 2026

Copy link
Copy Markdown
Author

Merge-ready check-in

This PR looks ready for maintainer review/merge when you have a moment.

  • Change: fix: runCollection stability and putCollection noauth
  • CI: green / passing
  • Review: (mergeable: MERGEABLE)

Happy to rebase or address any remaining feedback. Thanks for maintaining the project.

@garciasdos

garciasdos commented Jun 19, 2026

Copy link
Copy Markdown
Member

Thanks @advancedresearcharray! This PR bundles multiple independent fixes (addresses #107 stdio hardening, the Newman executor robustness change, and the putCollection noauth schema for #142). Please split it into one PR per fix so each can be reviewed, backported, and reverted independently. Note also that src/tools/putCollection.ts is auto-generated and synced from the Postman Public API specification, so the noauth change can't be merged via a direct edit here — we've noted it for the spec. The stdio and Newman executor changes are great candidates to land once split out. cc @akinard-postman.

@garciasdos garciasdos closed this Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akira28 akira28 Awaiting requested review from akira28
@rubenbaraut rubenbaraut Awaiting requested review from rubenbaraut rubenbaraut is a code owner
@alexander-rubia alexander-rubia Awaiting requested review from alexander-rubia alexander-rubia is a code owner
@akinard-postman akinard-postman Awaiting requested review from akinard-postman
@garciasdos garciasdos Awaiting requested review from garciasdos garciasdos is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@advancedresearcharray @garciasdos