Skip to content

Expose browser pool viewport - #78

Open
IlyaasK wants to merge 1 commit into
hypeship/browser-pool-data-source-warmup-configfrom
hypeship/browser-pool-data-source-viewport
Open

Expose browser pool viewport#78
IlyaasK wants to merge 1 commit into
hypeship/browser-pool-data-source-warmup-configfrom
hypeship/browser-pool-data-source-viewport

Conversation

@IlyaasK

@IlyaasK IlyaasK commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • expose computed nested viewport on kernel_browser_pool
  • preserve omitted viewport as a typed null object
  • require positive width/height and optional positive refresh rate
  • reject explicit null, malformed objects, missing dimensions, and invalid child values
  • update generated Registry docs

Scope

Additive v1 durable read state only. Chrome policy and live acceptance remain separate; no runtime/session fields are exposed.

Tests

Ran focused tests repeatedly, race tests, full short suite, vet, gofmt, module checks, Terraform example fmt, docs drift, and diff checks.

Coverage includes Framework nested-state roundtrip, typed-null omission, omitted refresh rate, malformed/null viewport, missing and non-positive dimensions, invalid refresh rate, and wrong JSON types.

Acceptance not run; live viewport payload behavior remains for the opt-in acceptance slice.


Note

Low Risk
Additive read-only data source field with validation and tests; no auth, writes, or runtime pool behavior changes.

Overview
Adds a computed nested viewport attribute to the kernel_browser_pool data source so Terraform can read durable pool browser window settings from the API.

When the API omits viewport, state uses a typed null object; when present, width and height must be positive and refresh_rate is optional but must be positive if sent. Malformed or invalid viewport payloads (explicit null, wrong types, missing dimensions, non-positive values) surface as read diagnostics instead of silent bad state. Registry docs for the data source are updated, with unit tests covering happy paths and rejection cases.

Reviewed by Cursor Bugbot for commit 3825585. Bugbot is set up for automated code reviews on this repo. Configure here.

@IlyaasK
IlyaasK requested review from Sayan- and tnsardesai and removed request for Sayan- and tnsardesai July 11, 2026 11:28
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from 027d9e8 to a2270fa Compare July 13, 2026 13:35
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from deb65f4 to df3b49e Compare July 13, 2026 13:35
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from a2270fa to a60a6b4 Compare July 13, 2026 14:32
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from df3b49e to 16097b4 Compare July 13, 2026 14:32
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from a60a6b4 to 9699f7f Compare July 13, 2026 15:39
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from 16097b4 to 8a4fc9c Compare July 13, 2026 15:39
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from 9699f7f to de2d434 Compare July 20, 2026 19:38
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from 8a4fc9c to da6aac2 Compare July 20, 2026 19:38
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from de2d434 to cd0ea6e Compare July 21, 2026 14:46
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch 3 times, most recently from 7c63d7e to 38e7266 Compare July 21, 2026 19:51
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from 0ac42d2 to 1bcac42 Compare July 21, 2026 20:15
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from 38e7266 to 1b81db1 Compare July 21, 2026 20:15
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-warmup-config branch from 1bcac42 to 9396869 Compare July 31, 2026 19:26
Read viewport configuration into a typed Terraform object. Reject malformed dimensions and preserve omitted viewport and refresh-rate values without partial state.
@IlyaasK
IlyaasK force-pushed the hypeship/browser-pool-data-source-viewport branch from 1b81db1 to 3825585 Compare July 31, 2026 19:28
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.

1 participant