Apply PR #110's Read index doc comment fix to v1 templates

[TakumaKurosawa]

Summary

The v1 templates/index.go.tpl hardcodes the doc comment on every generated Read{{ .FuncName }} as // Generated from unique index '...'., even when the underlying index is not unique. As a result, downstream users of the v1 generator see misleading doc comments on non-unique-index ReadXxx functions.

This is the same issue that was fixed for the v2 templates in #110 (merged 2022-12-07). That fix has not been backported to the v1 templates, and the v1 series is still being actively released (v0.8.0 and v0.8.1 in January 2026), so it remains worth fixing for v1 users.

Change

Mirror #110 exactly — drop the word unique from the Read{{ .FuncName }} doc comment so it reads // Generated from index '...'. regardless of uniqueness:

-// Generated from unique index '{{ .Index.IndexName }}'.
+// Generated from index '{{ .Index.IndexName }}'.
 func Read{{ .FuncName }}(ctx context.Context, db YORODB, keys spanner.KeySet) ([]*{{ .Type.Name }}, error) {

tplbin/templates.go and test/testmodels/**/*.yo.go are regenerated via make regen and make testdata-from-ddl to reflect the template change.

Scope

• Comment-only change. No runtime behavior change.
• Touches only the v1 paths: templates/index.go.tpl, tplbin/templates.go, and regenerated test/testmodels/**/*.yo.go.
• The v2/ tree is untouched (already fixed by Fix typo in index.go.tpl #110).

Test plan

• make regen (rebuild embedded templates)
• go build
• make testdata-from-ddl (regenerate v1 test models from DDL)
• go vet ./... clean
• gofmt -l clean
• make test was not run locally because it requires a running Spanner emulator and the diff is comment-only with no runtime behavior change. Happy to run it via the Spanner emulator if a reviewer would like to confirm.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[TakumaKurosawa]

@googlebot I signed it!

[TakumaKurosawa]

Friendly ping 🙏 @kazegusuri @sinmetal

This is a small, comment-only change that backports your #110 fix to the v1 templates (the Read{{ .FuncName }} doc comment still says "unique index" for non-unique indexes on v1). Since v1 is still actively released (v0.8.0 / v0.8.1 in Jan 2026), it seemed worth bringing v1 in line with v2.

The diff is the same one-word change as #110, with tplbin/templates.go and the test models regenerated via make regen / make testdata-from-ddl. CLA is signed and CI is green.

Would any maintainer have a moment to take a look? Happy to address any feedback, and I can run make test against the Spanner emulator if you'd like that confirmed.