<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Label this issue](https://contributors.gitlab.com/manage-issue?action=label&projectId=65271576&issueIid=2274) </details> <!--IssueSummary end--> Related to #2271. ## Summary Work item methods (`CreateWorkItem`, `GetWorkItem`, `ListWorkItems`, `UpdateWorkItem`) fail on **GitLab Community Edition (CE)** because the shared `workItemTemplate` in `workitems.go` unconditionally selects EE-only widget fields under `features`. On CE these fields are absent from the compiled GraphQL schema, producing `Field '...' doesn't exist on type 'WorkItemFeatures'` validation errors that fail the entire query. This is an **edition** problem (CE vs EE), **not** a tier problem (Free vs Premium/Ultimate). On any EE instance (including free GitLab.com namespaces) these fields exist and resolve to `null` when unavailable. They are absent only on CE (or EE versions predating a field). The switch must therefore key off `Metadata.Enterprise`. ## CE-absent fields to gate Confirmed against parent #2271 and the `features { ... }` block in `workItemTemplate`: - `color` (`color`, `textColor`) - `healthStatus` (`healthStatus`) - `iteration` (`iteration { id }`) - `status` (`status { name }`) - `weight` (`weight`) Fields that exist in CE and must stay unconditional: `assignees`, `hierarchy`, `labels`, `linkedItems`, `milestone`, `startAndDueDate`. ## Proposed solution ### 1. Cache the edition once per client with `sync.Once` Memoize the edition on `WorkItemsService` so `/metadata` is fetched at most once per client lifetime, using the existing `s.client.Metadata.GetMetadata()`. ```go type WorkItemsService struct { client *Client eeOnce sync.Once enterprise bool eeErr error } func (s *WorkItemsService) isEnterprise(options ...RequestOptionFunc) (bool, error) { s.eeOnce.Do(func() { md, _, err := s.client.Metadata.GetMetadata(options...) if err != nil { s.eeErr = err return } s.enterprise = md.Enterprise }) return s.enterprise, s.eeErr } ``` Note: ensure the `sync.Once`-bearing struct is never copied (always used as `*WorkItemsService`); keep `go vet` copylocks clean. ### 2. Gate the five fields in the template behind a bool Wrap each CE-absent block with `{{ if .Enterprise }}` inside `workItemTemplate`'s `features { ... }`: ```go {{- if .Enterprise }} color { color textColor } healthStatus { healthStatus } iteration { iteration { id } } status { status { name } } weight { weight } {{- end }} ``` ### 3. Pass the flag when executing the template All call sites currently execute with `nil` (or with `listWorkItemsQueryData{Decls, Args}`). Each must resolve the edition first and pass it through: - Add `Enterprise bool` to `listWorkItemsQueryData`. - Give `getWorkItemTemplate`, `createWorkItemTemplate`, and `updateWorkItemTemplate` a small data struct (or anonymous `struct{ Enterprise bool }`) to carry the flag. Each method gains a leading: ```go enterprise, err := s.isEnterprise(options...) if err != nil { return nil, nil, err } // pass enterprise into the template Execute(...) ``` ## Notes / trade-offs - **Response parsing**: `workItemFeaturesGQL.unwrap` already null-checks each widget pointer, so omitting the fields just leaves those `WorkItem` fields nil on CE. No parsing changes needed. - **Extra round-trip**: first work item call per client triggers one `/metadata` request; `sync.Once` keeps it to exactly one. - **Error propagation**: decision needed — fail fast if `/metadata` errors (recommended, makes cause visible) vs. degrade gracefully to CE-safe field set. ## Testing (required per AGENTS.md) - EE-path test asserting the five fields are present (existing `testdata/query_list_work_items.graphql` golden file encodes the EE shape and stays valid). - CE-path test: mock `/metadata` returning `enterprise: false`; assert the five fields are absent and the call succeeds. - Test that `sync.Once` calls `/metadata` only once across multiple work item calls. - Use GIVEN/WHEN/THEN Gherkin comments and `testify/assert`. Run `mise exec -- make reviewable` before committing. Interface signatures are unchanged (only unexported fields added), so mock regeneration is likely unnecessary but `make generate` should still run.