Skip to content

Add basic project view management - #2961

Open
zwick wants to merge 3 commits into
mainfrom
zwick-plan-project-v2-views
Open

Add basic project view management#2961
zwick wants to merge 3 commits into
mainfrom
zwick-plan-project-v2-views

Conversation

@zwick

@zwick zwick commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds basic ProjectV2 view lifecycle support to the consolidated Projects tools: list/get views plus create/update/delete with name, layout, and filter. Create supports visible fields by database ID or case-insensitive field name.

Why

Fixes #2954.

What changed

  • Added list_project_views and get_project_view with normalized output, pagination, and project-content IFC labels.
  • Added create_project_view, update_project_view, and delete_project_view; create uses the public REST endpoint and can resolve visible_field_names to the required database IDs.
  • Verify update/delete view ownership against the requested owner and project before mutation.
  • Added focused lifecycle tests, tool snapshots, Projects instructions, and generated README docs.

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed - extends the existing consolidated Projects tools with five methods.
  • New tool added

Prompts tested (tool changes only)

  • "List the views in my user project."
  • "Create a table view filtered to status:Todo with Status and Assignees visible."
  • "Get the new view, rename it, switch it to board layout, and filter to assignee:@me."
  • "Clear the view filter, then delete the view and list the views again."

Validated through the built stdio MCP server against the public APIs without preview headers, including visible_field_names resolution, ownership-mismatch rejection, and final cleanup/read-back.

Security / limits

  • No security or limits impact
  • Auth / permissions considered - existing read:project and project scopes are unchanged, and mutations verify the view belongs to the requested project.
  • Data exposure, filtering, or token/size limits considered - read results use project-content IFC labels and list pagination remains capped at 50.

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

Docs

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

@zwick
zwick force-pushed the zwick-plan-project-v2-views branch from 861109f to 8fb2181 Compare July 27, 2026 17:26
@zwick zwick linked an issue Jul 27, 2026 that may be closed by this pull request
@zwick zwick self-assigned this Jul 28, 2026
@zwick
zwick marked this pull request as ready for review July 28, 2026 13:40
@zwick
zwick requested a review from a team as a code owner July 28, 2026 13:40
Copilot AI review requested due to automatic review settings July 28, 2026 13:40

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.

Pull request overview

Adds complete basic ProjectV2 view lifecycle support to the consolidated Projects tools.

Changes:

  • Adds list/get/create/update/delete view operations with pagination and IFC labeling.
  • Supports layouts, filters, visible fields, and ownership verification.
  • Adds tests, snapshots, instructions, and generated documentation.
Show a summary per file
File Description
README.md Documents view parameters.
pkg/github/toolset_instructions.go Adds view workflow guidance.
pkg/github/projects.go Implements view lifecycle operations.
pkg/github/projects_v2_test.go Tests view behavior and authorization.
pkg/github/projects_test.go Verifies updated tool schemas.
pkg/github/minimal_types.go Adds normalized view output type.
pkg/github/__toolsnaps__/projects_write.snap Updates write schema snapshot.
pkg/github/__toolsnaps__/projects_list.snap Updates list schema snapshot.
pkg/github/__toolsnaps__/projects_get.snap Updates get schema snapshot.

Review details

  • Files reviewed: 9/9 changed files
  • Comments generated: 0
  • Review effort level: Medium

@zwick
zwick marked this pull request as draft July 28, 2026 13:49
zwick added 3 commits July 28, 2026 10:20
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c6f8ede6-efee-4191-900d-59a1bb0af000
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c6f8ede6-efee-4191-900d-59a1bb0af000
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c6f8ede6-efee-4191-900d-59a1bb0af000
@zwick
zwick force-pushed the zwick-plan-project-v2-views branch from cd30e93 to e519780 Compare July 28, 2026 14:26
@zwick
zwick marked this pull request as ready for review July 28, 2026 15:19
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.

Add basic Project view lifecycle operations to the Projects tools

2 participants