Uh oh!
There was an error while loading. Please reload this page.
Drop stale PLACEHOLDER annotations that shadow upstream descriptions - #5588
Merged
Conversation
Drop the checked-in annotations_openapi.yml: the schema generator now derives those annotations from .codegen/cli.json in memory on every run. Merge annotations.yml and annotations_openapi_overrides.yml into a single annotations.yml whose structure mirrors the bundle configuration tree. Docs absent from the file inherit from cli.json; entries override them. The generated jsonschema.json and jsonschema_for_docs.json are unchanged. Co-authored-by: Isaac
A `description: PLACEHOLDER` marks a field that has no documentation anywhere. It is added when a field first appears undocumented, but was never removed when cli.json later gained a description for the field. Because the annotations file overrides upstream docs in the merge, the stale marker shadowed the real description, and since PLACEHOLDER is skipped when assigning annotations, the schema emitted no description at all (e.g. `vector_search_endpoints.*.target_qps`). Strip such placeholders in newAnnotationHandler before the merge: the upstream description shows through, and because the pruned map is what syncWithMissingAnnotations writes back, the annotations file sheds the stale markers in the same run. Entries that carry other hand-authored fields (e.g. deprecation_message) lose only the placeholder. Fields that are genuinely undocumented keep their TODO marker, so the result is idempotent. TestRequiredAnnotationsForNewFields now also records deletes and updates during regeneration instead of nil-panicking on them, so any future non-idempotent rewrite of annotations.yml fails with a message instead of a panic. Co-authored-by: Isaac
Co-authored-by: Isaac
PR #5574 (the annotation consolidation this branch was stacked on) landed on main, superseding the local consolidation commit. Resolved conflicts by taking main's consolidation wholesale and re-applying the PLACEHOLDER-shadowing fix, adapted to main's restructured types: annotation.File is now map[string]TypeAnnotation{Self,Fields} and Descriptor dropped Preview and gained LaunchStage/EnumLaunchStages/EnumDescriptions. Regenerated jsonschema.json, jsonschema_for_docs.json, and annotations.yml. Co-authored-by: Isaac
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 08:25 — with 
GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 08:25 — with GitHub Actions
Inactive
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Replace the field-by-field enumeration with reflect.DeepEqual against a zero Descriptor, so adding a Descriptor field can no longer silently break the "is this entry droppable?" check after a stale placeholder is cleared. Co-authored-by: Isaac
Dropping the stale PLACEHOLDER markers restored upstream descriptions in jsonschema.json; pydabs-codegen reads that schema, so the generated Python model docstrings were stale and the validate-generated CI job failed. Regenerated via `task pydabs-codegen` (uv.lock churn reverted, as CI does). Co-authored-by: Isaac
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 08:47 — with GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 08:47 — with GitHub Actions
Inactive
janniklasrose
marked this pull request as ready for review
June 17, 2026 08:53
janniklasrose
enabled auto-merge
June 17, 2026 09:24
Collaborator
Integration test reportCommit: 9eca7d1
22 interesting tests: 14 SKIP, 7 RECOVERED, 1 flaky
Top 25 slowest tests (at least 2 minutes):
|
pietern
approved these changes
Jun 17, 2026
Uh oh!
There was an error while loading. Please reload this page.
…ssing-annotations Conflicts: NEXT_CHANGELOG.md
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 14:57 — with GitHub Actions
Inactive
janniklasrosetemporarily deployed
to
test-trigger-is
June 17, 2026 14:57 — with GitHub Actions
Inactive
Uh oh!
There was an error while loading. Please reload this page.
pietern added a commit
that referenced
this pull request
Jun 17, 2026
Pick up #5588's dropShadowingPlaceholders so cli.json descriptions (e.g. gen_ai_compute_task) flow into the schema instead of being shadowed by stale PLACEHOLDER markers. Also restore 0644 mode on the tagging files that genkit marked executable. Co-authored-by: Isaac
eng-dev-ecosystem-bot
commented
Jun 17, 2026
Collaborator
Integration test reportCommit: 522c859
541 interesting tests: 489 MISS, 33 FAIL, 9 KNOWN, 4 PANIC, 2 SKIP, 2 RECOVERED, 2 flaky
Top 50 slowest tests (at least 2 minutes):
|
deco-sdk-taggingBot
added a commit
that referenced
this pull request
Jun 24, 2026
## Release v1.5.0 ### CLI * `workspace export-dir` no longer aborts when a workspace object's name is not a legal local filename (e.g. a notebook named `New Notebook 2026-05-04 13:54:24` whose `:` is illegal on Windows). Such files are now exported under a sanitized name with a warning and the export completes ([#5171](#5171)). * `ssh connect` now opens an interactive `bash` login shell by default instead of the compute image's default `/bin/sh`, falling back gracefully when `bash` is unavailable. Passing an explicit remote command (`-- <cmd>`) is unaffected ([#5687](#5687)). * `ssh connect` interactive sessions now start in the user's workspace home folder (`/Workspace/Users/<email>`) instead of the OS home directory, falling back to the OS home when that folder is unavailable ([#5688](#5688)). ### Bundles * Add documentation for the common bundle resource fields `permissions`, `lifecycle`, and `grants` in the JSON schema, so they surface in editor completions and the docs. * `bundle run` now prints the modern job run URL (`/jobs/<id>/runs/<id>`) so that non-admin users permitted to view the run are taken to the run instead of the workspace homepage. * References to a registered model's `registered_model_id` now resolve under the direct engine, matching Terraform behavior ([#5621](#5621)). * Fix missing field descriptions in the bundle JSON schema for fields whose upstream API docs arrived after the field was first annotated (e.g. `vector_search_endpoints.*.target_qps`); stale placeholder markers no longer hide them ([#5588](#5588)). * Fix `bundle deploy --plan` dropping a `postgres_role`'s `role_id`, which caused the role to be recreated on the next deploy ([#5672](#5672)). * direct: Fix spurious cluster recreate when `apply_policy_default_values: true` is set ([#5693](#5693)). * direct: New 'deployment migrate' implementation that parses terraform state instead of fetching resources state from the backend ([#5399](#5399)). ### Dependency updates * Bump `github.com/databricks/databricks-sdk-go` from v0.141.0 to v0.147.0 ([#5636](#5636)). * Bump Terraform provider from v1.117.0 to v1.118.0 ([#5637](#5637)).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
description: PLACEHOLDERmarkers are now dropped when cli.json documents the field:newAnnotationHandlerstrips them before the merge, so the upstream description flows into the schema, and the sync step rewritesannotations.ymlwithout them in the same run.deprecation_message) lose only the placeholder.annotations.yml−484 lines (deletions only); both schema artifacts gain 128 description lines (23 of which replace a bare launch-stage tag).Why
A PLACEHOLDER means "no documentation anywhere". It is added when a field first appears undocumented, but nothing removed it once upstream docs arrived. The annotations file overrides upstream docs in the merge and PLACEHOLDER is skipped when assigning descriptions, so the schema ended up with no description at all — the stale marker hides the real text and then discards itself. 128 fields in the published schema were affected (e.g.
vector_search_endpoints.*.target_qps). The file cannot self-heal: a shadowed field never registers as missing, so the backlog only grows.This also fixes the bare
[Public Preview]prefixes from the launch-stage work (#5443): the prefix was there, but the description it should prefix was swallowed by this bug.Tests
TestRequiredAnnotationsForNewFieldsnow also records deletes/updates during regeneration (previously a nil-visitor panic) and asserts regeneration of the committed file is a no-op, enforcing idempotency.bundle/internal/schema+annotation+bundle/...unit tests, therefschemaacceptance test, andlintall pass; regeneration is byte-idempotent across runs.This pull request and its description were written by Isaac, an AI coding agent.