@johnstcn @mafredri

I've applied the solution we chatted yesterday, and it works well on the SQL level, but when it comes to model structures, sqlc converted all properties of GetWorkspaceRows to nullables (see CI). The dumb solution would be to coalesce all properties with "zero" values, but wouldn't it be too dirty?

EDIT:

Is it the moment when I need to go through the information_schema?

EDIT2:

I think I will go with prepending an extra technical row with defaults. I don't want anyone accidentally messed up the workspace data.

An ugly hack would be to ensure there is always one empty row of workspace and use inner join instead of left:

with filtered_workspaces_order as (
  select ...
  union all
  select '', '', ...
)

Maybe sqlc.embed could help too?

Whenever I've had issues with sqlc 'incorrectly' inferring a nullable type, I've always worked around it by adding explicit type casts.

Edit: you also need to explicitly name the columns so they line up correctly with GetWorkspaceRow otherwise you get struct fields like FwoID and so on.

Something like:

	fwo.id :: uuid as id,
	fwo.created_at :: timestamptz as created_at,
	fwo.updated_at :: timestamptz as updated_at,
	fwo.owner_id :: uuid as owner_id,
	fwo.organization_id :: uuid as organization_id,
	fwo.template_id :: uuid as template_id,
	fwo.deleted :: boolean as deleted,
	fwo.name :: text as name,
	fwo.autostart_schedule as autostart_schedule, -- nullable so no typecast
	fwo.ttl as ttl, -- nullable so no typecast
	fwo.last_used_at :: timestamptz as last_used_at,
	fwo.dormant_at as dormant_at, -- nullable so no typecast
	fwo.deleting_at as deleting_at, -- nullable so no typecast
	fwo.automatic_updates :: automatic_updates as automatic_updates,
	fwo.favorite :: boolean as favorite,
	fwo.template_name :: text as template_name,
	fwo.template_version_id :: uuid as template_version_id,
	fwo.template_version_name :: text as template_version_name,
	fwo.username :: text as username,
	fwo.latest_build_completed_at :: timestamptz as latest_build_completed_at,
	latest_build_canceled_at :: timestamptz as latest_build_canceled_at,
	latest_build_error :: text as latest_build_error,
	latest_build_transition :: workspace_transition as latest_build_transition

Also note that you will need to manually update the scan order of the GetAuthorizedWorkspaces query when you make any changes to GetWorkspaces

I might have sorted it out 👍 The query does not look great but works. I will work on tests and polishing. Thanks for guidance!

for reviewers: duplicated function, probably bad merge