[4/n][sl][github][gh stacks] update submit body for stacks by raydatray · Pull Request #1395 · facebook/sapling

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

raydatray · 2026-07-31T19:16:25Z

ctx

github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks

changes made

create_pull_request_title_and_body takes a stack_list flag; the stacked workflow omits the footer, other workflows are unchanged

test plan

updated mocks + test-ext-github-pr-submit-stacked.t

Stack created with Sapling. Best reviewed with ReviewStack.

meta-codesync · 2026-07-31T19:20:42Z

This pull request has been imported. If you are a Meta employee, you can view this in D114402698. (Because this pull request was imported automatically, there will not be any future comments.)

facebook-github-tools · 2026-08-13T18:54:29Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

### ctx github added native stacked pull requests, it would be nice to add native support to it for sapling! this stack lets `sl pr submit` and `sl pull` via a new `github.pr.workflow = stacked` mode ### changes made - `github_gh_cli.make_request` accepts custom headers (`-H`, needed for the `X-GitHub-Api-Version` preview header) - `_format_param` supports list values using the `gh api` repeated-field syntax (`pull_requests[]=101`) - an empty list is passed explicitly as `key[]` without a value (per the gh api manual) rather than dropped, since an empty array and a missing field can mean different things to an endpoint - mock_utils: `MockGitHubServer` now tracks which expectations were consumed; tests can opt in via `wrap_with_consumption_check` to fail when an expected request silently stops happening (closes the old TODO) ### test plan doctests in `_format_param` (registered in test-doctest.py, including the empty-array case), existing github .t suite. verified the empty-array wire format empirically: `gh api -F "pull_requests[]" --verbose` sends `{"pull_requests": []}`

### ctx basic plumbing for github native stacks REST API ### changes made - `gh_submit`: `StackDetails` dataclass + `get_stack_for_pull_request`, `create_stack`, `add_prs_to_stack`, `unstack`, all pinned to the `2026-03-10` preview API version - `unstack` returns the remaining stack when dissolution is partial (merged/queued diffs cannot be unstacked) so callers can react instead of assuming success - `update_pull_request` now takes `base: Optional[str]`: github rejects `updatePullRequest` mutations that include `baseRefName` for PRs in a native stack, so `base=None` uses a new mutation variant (`GRAPHQL_UPDATE_PULL_REQUEST_NO_BASE`) that only touches title/body ### test plan CI (doctest on `_parse_stack_from_dict`), and dogfooded on my own computer via a patch raydatray/rusty-mcrouter#198 <img width="905" height="468" alt="Screenshot 2026-07-31 at 3 20 04 PM" src="https://github.com/user-attachments/assets/bbe461a5-4128-4c75-b603-8ef4e012bb05" />

### ctx introduces the new native github stack workflow ### changes made - new `SubmitWorkflow.STACKED` variant, selected via `github.pr-workflow=stacked` - chains each PR's base to the head branch of the PR below it (like `single`), factored into `SubmitWorkflow.uses_chained_bases()` - hardened chained-base selection everywhere it happens (base-update loop, body rewrites, serial and placeholder creation): closed/merged diffs are skipped when picking the base below (their head branches would break the chain), and forks never chain (fork head branches cannot be bases on the upstream repo, so fall back to the default branch) ### test plan - new test-ext-github-pr-submit-stacked.t (initial submit) - test-ext-github-pr-submit-closed.t: open diffs stacked on a closed one chain past it to main - test-ext-github-pr-submit-placeholder-issue.t: placeholder strategy on a fork creates diffs against the upstream default branch

### ctx github renders native stacks in the PR UI itself so the sapling footer would be redundant - lets remove it when submitting via native stacks ### changes made - `create_pull_request_title_and_body` takes a `stack_list` flag; the stacked workflow omits the footer, other workflows are unchanged ### test plan updated mocks + test-ext-github-pr-submit-stacked.t

facebook-github-tools · 2026-08-13T19:13:50Z

@raydatray has updated the pull request. You must reimport the pull request before landing.

meta-claBot added the CLA Signed label Jul 31, 2026

raydatray mentioned this pull request Jul 31, 2026
Support Github Native Stack PRs #1390
Open