fixesA

common/proto/table_definition.proto

@@ -542,8 +542,8 @@ message ColumnBehavior {
   // This is not an authorization boundary; data APIs still expose the column.
   bool hidden_from_forms = 6;
 
-  // Current public aliases for stable option values. Identity aliases equal the
+  // Current public option names. For a renamed option, both fields contain its
-  // canonical value. Once renamed, public writes must use the current alias.
+  // alias; the stored value is never disclosed. Public writes use the alias.
   repeated OptionValueAlias option_value_aliases = 7;
 }
 
@@ -581,7 +581,8 @@ message SetColumnPresentationResponse {
   int64 row_version = 4;
 }
 
-// One stable machine value and its optional user-visible/input alias.
+// One public option name. `value` identifies the option by its current public
+// alias and `alias` is its desired public alias; stored values remain internal.
 message OptionValueAlias {
   string value = 1;
   string alias = 2;

common/src/proto/komp_ac.table_definition.rs

@@ -563,8 +563,8 @@ pub struct ColumnBehavior {
     /// This is not an authorization boundary; data APIs still expose the column.
     #[prost(bool, tag = "6")]
     pub hidden_from_forms: bool,
-    /// Current public aliases for stable option values. Identity aliases equal the
+    /// Current public option names. For a renamed option, both fields contain its
-    /// canonical value. Once renamed, public writes must use the current alias.
+    /// alias; the stored value is never disclosed. Public writes use the alias.
     #[prost(message, repeated, tag = "7")]
     pub option_value_aliases: ::prost::alloc::vec::Vec<OptionValueAlias>,
 }
@@ -621,7 +621,8 @@ pub struct SetColumnPresentationResponse {
     #[prost(int64, tag = "4")]
     pub row_version: i64,
 }
-/// One stable machine value and its optional user-visible/input alias.
+/// One public option name. `value` identifies the option by its current public
+/// alias and `alias` is its desired public alias; stored values remain internal.
 #[derive(serde::Serialize, serde::Deserialize)]
 #[derive(Clone, PartialEq, Eq, Hash, ::prost::Message)]
 pub struct OptionValueAlias {

komp-app/src/grpc.rs

@@ -37,6 +37,7 @@ use common::proto::komp_ac::table_definition::{
     GetTableCatalogResponse, ListColumnTypesResponse, PostTableDefinitionRequest,
     ProfileTreeResponse, PutTableDefinitionRequest, PutTableDefinitionResponse,
     SetColumnFormVisibilityRequest, SetColumnFormVisibilityResponse,
+    SetColumnOptionAliasesRequest, SetColumnOptionAliasesResponse,
     SetColumnPresentationRequest, SetColumnPresentationResponse,
     TableDefinitionResponse, table_definition_client::TableDefinitionClient,
 };
@@ -567,6 +568,19 @@ impl GrpcClient {
         Ok(response.into_inner())
     }
 
+    pub async fn set_column_option_aliases(
+        &mut self,
+        request: SetColumnOptionAliasesRequest,
+    ) -> Result<SetColumnOptionAliasesResponse> {
+        let tonic_request = self.authenticated_request(request)?;
+        let response = self
+            .table_definition_client
+            .set_column_option_aliases(tonic_request)
+            .await
+            .context("gRPC SetColumnOptionAliases call failed")?;
+        Ok(response.into_inner())
+    }
+
     pub async fn set_column_form_visibility(
         &mut self,
         request: SetColumnFormVisibilityRequest,