Skip to content

Stop relying on immutable and extension on Postgres parameters - #1461

Merged
ayrton merged 2 commits into
mainfrom
ayrton/drop-parameter-immutable
Oct 5, 2026
Merged

ayrton merged 2 commits into
mainfrom
ayrton/drop-parameter-immutable

Conversation

@ayrton

@ayrton ayrton commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

The API is removing immutable and extension from Postgres parameters. This stops the CLI from relying on either field:

  • pscale branch resize --parameters validates against the changeable parameters only (internal=false), so an internal parameter now fails the same way an unknown one does, with a message pointing to pscale branch parameters list --internal=false.
  • pscale branch parameters list drops the immutable column. --internal already filters on the server.
  • PostgresParameter and PostgresBouncerParameter drop the two fields.

The API is dropping both fields from Postgres parameters, as it already does for
Neki. resize asks the catalog for changeable parameters only, so internal ones
fail the same way unknown ones do, and parameters list drops the immutable
column; --internal still filters on the server.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@ayrton
ayrton marked this pull request as ready for review October 5, 2026 08:14
@ayrton
ayrton requested a review from a team as a code owner October 5, 2026 08:14

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Risk: low. Approved. Cursor Bugbot passed with no findings, Cursor Security Agent was not running, and this is a narrow parameter-catalog change with no human review required.

Open in Web View Automation 

Sent by Cursor Approval Agent: cli-reviewer

@ayrton
ayrton enabled auto-merge October 5, 2026 08:27
Also include --org in the hint printed for unknown parameters.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@ayrton
ayrton merged commit 828caae into main Oct 5, 2026
4 checks passed
@ayrton
ayrton deleted the ayrton/drop-parameter-immutable branch October 5, 2026 12:41
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