Skip to content

feat(copilot): open the table on the view table_views just wrote - #7166

Merged
waleedlatif1 merged 20 commits into
stagingfrom
feat/let-mothership-edit-views
Sep 3, 2026
Merged

waleedlatif1 merged 20 commits into
stagingfrom
feat/let-mothership-edit-views

Conversation

@j15z

@j15z j15z commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Connect the existing table_views write operations to the resource panel so it opens on the view they affected
  • Persist the selected view with each chat resource so reopening a chat restores the same table view
  • Restore the selected view immediately when switching away from a chat and back, including when refreshed resource metadata arrives after the table mounts
  • Serialize table-view default changes and chat-resource persistence so concurrent updates cannot overwrite newer state
  • Keep table-view validation, stream descriptors, resource contracts, and generated mothership contracts aligned

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

  • bun run lint
  • bun run lint:check
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • bun run check:audits — 44 audits passing
  • Focused Vitest coverage — 7 files and 90 tests passing
  • Tested manually in Chrome by selecting a non-default table view, switching chats, and returning to confirm the selected view is restored

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

Not included; this changes restored selection behavior without changing the table's visual design.

Companion: simstudioai/mothership#461

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants