feat(bigtable): modularize channel priming behind a ChannelPrimer interface - #20027

Resolved conflict in bigtable/internal/transport/channel_pool_factory.go:
upstream PR googleapis#20027 replaced WithInstanceName / WithAppProfile /
WithFeatureFlagsMetadata with a single WithChannelPrimer option built
from a pingAndWarmChannelPrimer. Updated the basePoolOpts helper to
accept that primer (dropping the old three args) and updated
CreateSessionChannelPool to build its own primer the same way — the
session pool channels still use PingAndWarm for per-connection priming;
only the Direct Access compatibility decision moves to the
GetClientConfiguration-based checker.

…erface (port of googleapis#20027)

Hand-port of upstream 5214ab7 into the sessionz branch. Cherry-pick
failed on layout drift for the same reason as PR googleapis#19987 — the top-level
bigtable/channel_pool_factory.go doesn't exist upstream — so the diff
was reproduced by hand.

Mirrors the pluggable shape that the DAC extract established:

- New internal/transport/channel_primer.go: ChannelPrimer interface with
  a single Prime(ctx, *BigtableConn) error method. Today's default
  implementation, pingAndWarmChannelPrimer, owns the (instance name,
  app profile, feature-flag metadata) tuple and delegates to
  BigtableConn.Prime. Constructor exported as NewPingAndWarmChannelPrimer
  so the top-level bigtable package can wire it.
- New internal/transport/channel_primer_test.go: covers (a) primer
  delegates PingAndWarm with configured feature-flag metadata,
  (b) connectionFactory skips priming entirely when primer is nil.
- connpool.go: add WithChannelPrimer option and channelPrimer field on
  BigtableChannelPool. Drop WithFeatureFlagsMetadata and the featureFlagsMD
  field. Keep WithInstanceName / WithAppProfile — the sessionz
  ChannelPoolSnapshot still reads them for debugview channelz/configz
  identity. connectionFactory now carries a ChannelPrimer instead of
  three individual fields; primeWithRetry returns immediately when the
  primer is nil so the pool can be used without priming.
- direct_access_checker.go: pingAndWarmDirectAccessChecker now takes a
  ChannelPrimer instead of (instance, appProfile, featureFlagsMD) — both
  CheckCompatibility and probeSingleEndpoint route their Prime through
  it. NewPingAndWarmDirectAccessChecker signature updated accordingly.
- bigtable/channel_pool_factory.go: construct one ChannelPrimer and
  share it with both the pool (via WithChannelPrimer) and the checker,
  eliminating the three-arg drift between them.

Test updates in connpool_test.go: poolOpts() adds the primer,
NilDirectAccessChecker case takes a primer to isolate the checker gate,
TestConnectionFactory factory struct uses the primer field, and the
five TestDirectAccessLogic subtests construct a primer for their DA
checker.

Verified: go build ./... , go vet ./... , and
go test ./internal/transport/... ./debugview/ -count=1 -short all pass.

Take upstream/main's versions of both files. Fork's ports of googleapis#20027
and googleapis#19987 exported the constructors (NewPingAndWarm...) and made
them return interfaces; upstream landed them unexported (newPingAndWarm...)
returning concrete types. Upstream's channel_pool_factory.go calls the
unexported forms, so restoring these files re-aligns the trio.

Downstream break to fix next: fork's internal/session and
internal/transport/session_* code that called the uppercase exported
constructors will now need to be updated to the unexported forms or
to route through the interface accessors upstream provides.

Restructures the spec-driven review deck from a 5-slide feature tour
into a 4-slide 'why we moved from one-shot prompting to specs' story:

- Slide 1 — one-shot prompting definition + the three PRs that shipped
  this way (googleapis#19987 DirectAccessChecker, googleapis#20027 ChannelPrimer, googleapis#20099
  client-side-metrics decouple). Common shape: extract one implicitly-
  unary abstraction into an interface.
- Slide 2 — Jetstream is 30k+ LOC; one-shot breaks. Introduces the five
  spec files with verified invariant counts (10 / 4 / 5 / 3 / 12+PartC)
  and one illustrative rule per spec.
- Slide 3 — SESSION_COMPONENT_SPEC.md tour: Part A (7-layer descriptive
  map), Part B (12 boundary MUST-rules with grep patterns), Part C
  (ownership matrix excerpts).
- Slide 4 — three prompt sizes with real examples from this branch:
  (1) simple refactor — activeVRPC/casActiveVRPC accessor extraction;
  (2) logic addition — adaptive session-creation throttler + how the
  spec invariants (POOL #5, CLIENT #3, B6) chain together;
  (3) big feature — unified debugview/ (7 z-pages behind one Handler)
  and how B3, POOL #4, B10 crystallized during that refactor.
  Plus reviewer-agent flow and PASS/VIOLATION/AMBIGUOUS semantics.

CSS and navigation unchanged. Counter reflects 4 slides. Companion
specs-deck.md not updated in this commit — will follow.

🤖 I have created a release *beep* *boop*
---


##
[1.51.0](bigtable/v1.50.0...bigtable/v1.51.0)
(2026-07-23)


### Features

* **bigtable:** Add ChainInterceptors and RetryingVRpc for vRPC pipeline
([#20185](#20185))
([c7a832a](c7a832a))
* **bigtable:** Add ClientConfigurationManager
([#19986](#19986))
([3a8f927](3a8f927))
* **bigtable:** Add debug tag counter (recordDebugTag / assertDebugTag)
([#20114](#20114))
([3c97590](3c97590))
* **bigtable:** Add lazyPool helper for on-demand session pool opening
([#20182](#20182))
([f6ae3fb](f6ae3fb))
* **bigtable:** Add PeakEwma continuous time-decay latency tracker
([#20187](#20187))
([9d124ef](9d124ef))
* **bigtable:** Add PoolSizer for server-driven session pool capacity
([#20189](#20189))
([57ebbeb](57ebbeb))
* **bigtable:** Add session package with SessionClient + SessionTableAPI
interfaces
([#20180](#20180))
([4b82fd2](4b82fd2))
* **bigtable:** Add Session primitives (AttemptOutcome, vRPC ctx,
msgtype)
([#20116](#20116))
([e1011e2](e1011e2))
* **bigtable:** Add Session state enum
([#19981](#19981))
([0748972](0748972))
* **bigtable:** Add SessionThrottler / AdaptiveSessionThrottler for
OpenSession pacing
([#20184](#20184))
([02e3c6d](02e3c6d))
* **bigtable:** Add SessionThrottler / AdaptiveSessionThrottler for
OpenSession pacing
([#20184](#20184))
([29be83e](29be83e))
* **bigtable:** Add sessionTracer for per-Session lifecycle + vRPC
metrics
([#20190](#20190))
([a466345](a466345))
* **bigtable:** Enable new auth library and JWT for instance admin
client
([#20013](#20013))
([21c4a44](21c4a44))
* **bigtable:** Modularize channel priming behind a ChannelPrimer
interface
([#20027](#20027))
([5214ab7](5214ab7))
* **bigtable:** Modularize Direct Access compatibility check
([#19987](#19987))
([a25e93d](a25e93d))
* **o11y:** Regenerate clients for LRO tracing
([#20107](#20107))
([779074e](779074e))


### Bug Fixes

* **bigtable:** Default cluster/zone in toOtelMetricAttrs to avoid
Monitoring reject
([#20178](#20178))
([14493f4](14493f4))
* **bigtable:** Eliminate stats-handler MD race in internal/metrics
tracer
([#20158](#20158))
([c387066](c387066))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>