chore: adopt released protoyaml for SHOW PLAN NODE canonical output

[apstndb]

Summary

Migrates SHOW PLAN NODE from the in-tree spannerplan/protoyaml marshalers to the released standalone module github.com/apstndb/protoyaml v0.1.0 (canonical protojson-over-YAML).

github.com/apstndb/spannerplan stays as a dependency; only the protoyaml subpackage import moves to the standalone module.

What changes for users

SHOW PLAN NODE output switches dialect from an accidental proto-reflection format (snake_case proto field names, numeric enums) to canonical protojson-over-YAML (camelCase, named enums) — matching the field names used in gcloud output and the official query plan visualizer JSON.

The layout is identical: all-scalar sub-maps still render as compact flow ({...}) via protoyaml.WithFlowLeafCollections(). Root-level scalar-only nodes stay as block documents (e.g. index: 1\nkind: RELATIONAL\ndisplayName: Child\n), not inline flow.

structpb data keys (the contents of execution_stats, metadata, etc.) are unchanged — those come from the plan payload, not proto field names.

Before / after

Before (proto-reflection dialect):

index: 1
kind: 1
display_name: Unit Relation
child_links:
- {child_index: 2}
metadata: {execution_method: Row}
execution_stats:
  cpu_time: {total: "0", unit: msecs}

After (canonical protojson-over-YAML):

index: 1
kind: RELATIONAL
displayName: Unit Relation
childLinks:
- {childIndex: 2}
metadata: {execution_method: Row}
executionStats:
  cpu_time: {total: "0", unit: msecs}

Note how the proto field names change (display_name -> displayName, kind: 1 -> kind: RELATIONAL) while the structpb-sourced metadata / execution_stats data keys stay as-is.

Implementation

• go get github.com/apstndb/protoyaml@v0.1.0 && go mod tidy.
• Swap the import github.com/apstndb/spannerplan/protoyaml -> github.com/apstndb/protoyaml.
• Replace the custom option builder call with protoyaml.Marshal(planNode, protoyaml.WithFlowLeafCollections()).
• Delete ~50 lines of custom goccy/go-yaml marshalers (getGlobalOpts, getStructOpts, hasCompound) and now-unused imports.
• Update TestShowPlanNodeStatement_Execute golden strings to the canonical output.

Net: 13 insertions, 59 deletions.

Validation

• go build ./..., go vet ./... clean.
• make check green (test + lint + fmt-check; golangci-lint 0 issues, formatting correct).

🤖 Generated with Claude Code

https://claude.ai/code/session_01Du9UR1LPdSXk4wAZr4Nf4g

[gemini-code-assist]

Code Review

This pull request replaces the internal protoyaml package and custom YAML marshaling logic with the external github.com/apstndb/protoyaml library. This simplifies statements_explain_describe.go by removing several custom marshaling helper functions. The test cases in statements_explain_describe_test.go have been updated to match the new YAML output format, which uses camelCase keys and string representations for enums. I have no additional feedback to provide as the changes are clean and the tests have been appropriately updated.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[github-actions]

Code Metrics Report

📊 View detailed coverage report (available for 7 days)

	main (53bd54f)	#784 (d97b778)	+/-
Coverage	73.2%	73.1%	-0.1%
Code to Test Ratio	1:1.3	1:1.3	+0.0
Test Execution Time	1m57s	2m1s	+4s

Details

  |                     | main (53bd54f) | #784 (d97b778) |  +/-  |
  |---------------------|----------------|----------------|-------|
- | Coverage            |          73.2% |          73.1% | -0.1% |
  |   Files             |             93 |             93 |     0 |
  |   Lines             |           7933 |           7917 |   -16 |
- |   Covered           |           5808 |           5794 |   -14 |
+ | Code to Test Ratio  |          1:1.3 |          1:1.3 |  +0.0 |
  |   Code              |          18517 |          18472 |   -45 |
  |   Test              |          25146 |          25146 |     0 |
- | Test Execution Time |          1m57s |           2m1s |   +4s |

Code coverage of files in pull request scope (63.2% → 61.7%)

Files	Coverage	+/-	Status
internal/mycli/statements_explain_describe.go	61.7%	-1.5%	modified

---

Reported by octocov