Skip to content

Add visible fields to project views - #2988

Open
zwick wants to merge 2 commits into
zwick-plan-project-v2-viewsfrom
zwick-add-project-view-visible-fields
Open

Add visible fields to project views#2988
zwick wants to merge 2 commits into
zwick-plan-project-v2-viewsfrom
zwick-add-project-view-visible-fields

Conversation

@zwick

@zwick zwick commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds ordered visible-field configuration to Project view create/update and read-back. This PR is stacked on #2961 and is blocked on that PR.

Why

The stable GraphQL API now supports view configuration.visibleFieldIds without preview headers.

What changed

  • Resolve existing numeric field IDs or names to specialized Relay IDs for create/update.
  • Preserve omitted configuration, support explicit-empty reset, and return ordered visible fields from view reads and mutations.

MCP impact

  • Tool schema or behavior changed
  • No tool or API changes
  • New tool added

projects_write grew by 202 bytes / 48 tokens (o200k_base); no parameters were added.

Prompts tested (tool changes only)

  • "Create a table view showing Teams then Status."
  • "Reorder this view to Status then Teams, preserve fields on rename, then reset it."

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Uses existing Projects permissions and returns only field database IDs already exposed by Projects tools.

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Also passed toolsnap and docs generation checks. Live disposable-resource dogfood covered create/get/list, specialized multi-select resolution, reorder, omission preservation, reset, and verified cleanup without preview headers.

Docs

  • Not needed
  • Updated (README / docs / examples)

@zwick zwick self-assigned this Aug 4, 2026
@zwick
zwick marked this pull request as ready for review August 6, 2026 16:18
@zwick
zwick requested a review from a team as a code owner August 6, 2026 16:18
Copilot AI balanced review requested due to automatic review settings August 6, 2026 16:18

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@zwick
zwick force-pushed the zwick-add-project-view-visible-fields branch from d7db5ca to bbf2398 Compare August 7, 2026 01:56
@zwick
zwick force-pushed the zwick-add-project-view-visible-fields branch from bbf2398 to 5b75042 Compare August 7, 2026 12:44
@zwick
zwick marked this pull request as draft August 7, 2026 13:16
@zwick
zwick force-pushed the zwick-add-project-view-visible-fields branch from 5b75042 to 8d0f208 Compare August 7, 2026 13:35
@zwick
zwick marked this pull request as ready for review August 7, 2026 13:58
tommaso-moro
tommaso-moro previously approved these changes Aug 7, 2026
zwick and others added 2 commits August 7, 2026 13:34
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 1421a5d5-fdce-4c0e-9528-56d555ec30d4
Reject roadmap layouts before enumerating project fields in both the
create and update paths, and verify view ownership before resolving
visible fields on update, so rejected requests no longer pay for a
paginated field listing.

Skip the follow-up filter mutation when the filter is explicitly null,
since a new view has no filter to clear, and include the created view ID
when cleanup after a failed filter mutation also fails so the caller can
recover the orphaned view.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@zwick
zwick force-pushed the zwick-add-project-view-visible-fields branch from 8d0f208 to 45526bd Compare August 7, 2026 17:37
@zwick

zwick commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Rebased against the changed PR in this stack

@zwick
zwick requested a review from tommaso-moro August 7, 2026 17:42
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.

3 participants