Fix doctests for protobuf 3.x API and add integration test docs

- Update MutateRow, MutateRows, CheckAndMutateRow doctests to use
  protobuf 3.x oneof enum variant syntax instead of field assignment
- Add integration testing section to CLAUDE.md with GCP project details
- Add *.json to .gitignore to protect credentials

Author: durch

.beads/issues.jsonl

@@ -1,3 +1,4 @@
+{"id":"rust-bigtable-30c","title":"Fix doctests for protobuf 3.x API","description":"Why: 3 doctests fail - MutateRow, MutateRows, CheckAndMutateRow use old delete_from_row field syntax instead of mutation oneof. Done when: cargo test -- --ignored passes.","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-12-10T00:10:09.897449+01:00","updated_at":"2025-12-10T00:12:18.386935+01:00","closed_at":"2025-12-10T00:12:18.386935+01:00","close_reason":"Fixed 3 doctests for protobuf 3.x API compatibility"}
 {"id":"rust-bigtable-3qg","title":"Update protobuf API usage (RepeatedField -\u003e Vec, getter changes)","description":"Why: protobuf 3.x removed RepeatedField, changed getter prefixes. Done when: all src files use Vec and new getter names.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-12-09T23:19:20.609494+01:00","updated_at":"2025-12-09T23:25:32.573329+01:00","closed_at":"2025-12-09T23:25:32.573329+01:00","close_reason":"Updated protobuf API: RepeatedField-\u003eVec, MessageFull trait, direct field access","dependencies":[{"issue_id":"rust-bigtable-3qg","depends_on_id":"rust-bigtable-nko","type":"blocks","created_at":"2025-12-09T23:20:08.478991+01:00","created_by":"daemon"},{"issue_id":"rust-bigtable-3qg","depends_on_id":"rust-bigtable-k8t","type":"blocks","created_at":"2025-12-09T23:20:16.756889+01:00","created_by":"daemon"}]}
 {"id":"rust-bigtable-6u0","title":"Update goauth API (get_token_with_creds -\u003e get_token_blocking)","description":"Why: goauth 0.17 renamed token retrieval function. Done when: utils.rs uses get_token_blocking or async get_token.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-12-09T23:19:33.88726+01:00","updated_at":"2025-12-09T23:23:20.639651+01:00","closed_at":"2025-12-09T23:23:20.639651+01:00","close_reason":"Updated get_token_with_creds to get_token_blocking","dependencies":[{"issue_id":"rust-bigtable-6u0","depends_on_id":"rust-bigtable-nko","type":"blocks","created_at":"2025-12-09T23:20:08.546442+01:00","created_by":"daemon"}]}
 {"id":"rust-bigtable-i1x","title":"Replace rustc-serialize with base64 crate","description":"Why: rustc-serialize is deprecated. Done when: base64 encoding uses base64 crate, rustc-serialize removed from deps.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-12-09T23:19:10.820357+01:00","updated_at":"2025-12-09T23:23:20.610896+01:00","closed_at":"2025-12-09T23:23:20.610896+01:00","close_reason":"Replaced rustc_serialize::base64 with base64 crate","dependencies":[{"issue_id":"rust-bigtable-i1x","depends_on_id":"rust-bigtable-nko","type":"blocks","created_at":"2025-12-09T23:20:08.445798+01:00","created_by":"daemon"}]}

.gitignore

@@ -3,3 +3,7 @@ Cargo.lock
 /pk.pem
 .idea*
 src/protos
+
+# Credentials - never commit
+*.json
+!dummy_credentials_file_for_tests.json

CLAUDE.md

@@ -52,6 +52,35 @@ cargo check
 
 ### Dependencies
 - `goauth` / `smpl_jwt`: Google OAuth2 / JWT authentication
-- `protobuf` / `protobuf-json-temp`: Protobuf message handling and JSON conversion
+- `protobuf` / `protobuf-json-mapping`: Protobuf message handling and JSON conversion
 - `curl`: HTTP client
 - `serde_json`: JSON serialization
+
+## Integration Testing
+
+### GCP Resources
+- **Project ID**: `gen-lang-client-0421059902`
+- **Instance**: `test-inst`
+- **Table**: `my-table`
+- **Column Family**: `cf1`
+
+### Setup Commands
+```bash
+# Create instance
+gcloud bigtable instances create test-inst \
+  --project=gen-lang-client-0421059902 \
+  --cluster=test-cluster \
+  --cluster-zone=us-central1-c \
+  --display-name="Test Instance"
+
+# Create table with column family
+cbt -project=gen-lang-client-0421059902 -instance=test-inst createtable my-table
+cbt -project=gen-lang-client-0421059902 -instance=test-inst createfamily my-table cf1
+```
+
+### Credentials
+Requires a service account JSON key with `roles/bigtable.user` permission.
+- **Credentials file**: `Rust Bigtable IAM Admin.json` (gitignored)
+- **Service account**: `rustbigtable@gen-lang-client-0421059902.iam.gserviceaccount.com`
+
+Usage: `get_auth_token("Rust Bigtable IAM Admin.json", true)`

src/method.rs

@@ -120,7 +120,7 @@ method!(SampleRowKeys, SampleRowKeysRequest, false);
 /// use bt::request::BTRequest;
 /// use bt::utils::*;
 /// use bt::method::{BigTable, MutateRow};
-/// use bt::protos::data::Mutation;
+/// use bt::protos::data::{Mutation, mutation};
 /// use bt::error::BTErr;
 ///
 /// fn wrapper() -> Result<(), BTErr> {
@@ -132,11 +132,11 @@ method!(SampleRowKeys, SampleRowKeysRequest, false);
 ///
 ///     let row_key = encode_str("r1");
 ///
-///     let mut mutation = Mutation::new();
-///     mutation.delete_from_row = Some(Default::default()).into();
+///     let mut m = Mutation::new();
+///     m.mutation = Some(mutation::Mutation::DeleteFromRow(Default::default()));
 ///
 ///     req.method.payload_mut().row_key = row_key;
-///     req.method.payload_mut().mutations.push(mutation);
+///     req.method.payload_mut().mutations.push(m);
 ///
 ///     let response = req.execute(&get_auth_token("credentials.json", true)?)?;
 ///     println!("{}", serde_json::to_string_pretty(&response)?);
@@ -153,7 +153,7 @@ method!(MutateRow, MutateRowRequest, true);
 /// use bt::request::BTRequest;
 /// use bt::utils::*;
 /// use bt::method::{BigTable, MutateRows};
-/// use bt::protos::data::Mutation;
+/// use bt::protos::data::{Mutation, mutation};
 /// use bt::protos::bigtable::mutate_rows_request;
 /// use bt::error::BTErr;
 ///
@@ -166,12 +166,12 @@ method!(MutateRow, MutateRowRequest, true);
 ///
 ///     let row_key = encode_str("r1");
 ///
-///     let mut mutation = Mutation::new();
-///     mutation.delete_from_row = Some(Default::default()).into();
+///     let mut m = Mutation::new();
+///     m.mutation = Some(mutation::Mutation::DeleteFromRow(Default::default()));
 ///
 ///     let mut entry = mutate_rows_request::Entry::new();
 ///     entry.row_key = row_key;
-///     entry.mutations.push(mutation);
+///     entry.mutations.push(m);
 ///
 ///     req.method.payload_mut().entries.push(entry);
 ///
@@ -190,7 +190,7 @@ method!(MutateRows, MutateRowsRequest, true);
 /// use bt::request::BTRequest;
 /// use bt::utils::*;
 /// use bt::method::{BigTable, CheckAndMutateRow};
-/// use bt::protos::data::{RowFilter, Mutation};
+/// use bt::protos::data::{RowFilter, Mutation, mutation, row_filter};
 /// use bt::error::BTErr;
 ///
 /// fn wrapper() -> Result<(), BTErr> {
@@ -203,14 +203,14 @@ method!(MutateRows, MutateRowsRequest, true);
 ///     let row_key = encode_str("r1");
 ///
 ///     let mut predicate_filter = RowFilter::new();
-///     predicate_filter.set_pass_all_filter(true);
+///     predicate_filter.filter = Some(row_filter::Filter::PassAllFilter(true));
 ///
-///     let mut mutation = Mutation::new();
-///     mutation.delete_from_row = Some(Default::default()).into();
+///     let mut m = Mutation::new();
+///     m.mutation = Some(mutation::Mutation::DeleteFromRow(Default::default()));
 ///
 ///     req.method.payload_mut().row_key = row_key;
 ///     req.method.payload_mut().predicate_filter = Some(predicate_filter).into();
-///     req.method.payload_mut().true_mutations.push(mutation);
+///     req.method.payload_mut().true_mutations.push(m);
 ///
 ///     let response = req.execute(&get_auth_token("credentials.json", true)?)?;
 ///     println!("{}", serde_json::to_string_pretty(&response)?);