refactor(quarto-core): rename from_config_value to identity_map_from_…

…meta

The free function in `attribution/types.rs` parses
`meta.attribution.identities` into an `IdentityMap`. The old name
read as a generic "from any ConfigValue" converter and collided in
search results with the unrelated `Navbar::from_config_value` /
`Sidebar::from_config_value` / `PageNavigation::from_config_value`
methods elsewhere in the crate. The new name says what the function
actually returns and what it reads from.

crates/quarto-core/src/attribution/mod.rs

@@ -53,5 +53,5 @@ pub use source::{AttributionSource, AttributionSourceProvider};
 pub use types::{
     AttributionData, AttributionHit, AttributionMap, AttributionRecord, AttributionRun, Identity,
     IdentityMap, TransportAttributionData, TransportAttributionRun, format_supports_attribution,
- from_config_value,
+ identity_map_from_meta,
 };

crates/quarto-core/src/attribution/types.rs

@@ -193,7 +193,7 @@ pub fn format_supports_attribution(format: &Format) -> bool {
 /// uses them only as lookup keys and preserves the provider's
 /// pointer-equal key on collision, so the writer-side
 /// `Arc::ptr_eq` interning invariant is not weakened.
-pub fn from_config_value(meta: &quarto_pandoc_types::ConfigValue) -> IdentityMap {
+pub fn identity_map_from_meta(meta: &quarto_pandoc_types::ConfigValue) -> IdentityMap {
     let Some(identities) = meta.get("attribution").and_then(|v| v.get("identities")) else {
         return IdentityMap::new();
     };

crates/quarto-core/src/transforms/attribution_generate.rs

@@ -36,7 +36,7 @@ use std::sync::Arc;
 use quarto_pandoc_types::pandoc::Pandoc;
 
 use crate::Result;
-use crate::attribution::{AttributionData, format_supports_attribution, from_config_value};
+use crate::attribution::{AttributionData, format_supports_attribution, identity_map_from_meta};
 use crate::render::RenderContext;
 use crate::transform::AstTransform;
 use crate::transforms::is_feature_disabled;
@@ -97,7 +97,7 @@ impl AstTransform for AttributionGenerateTransform {
         // non-colliding user keys (an actor named in YAML but with
         // no runs in this document is invisible at the writer and
         // would be dead weight in the map).
-        for (user_key, user_id) in from_config_value(&ast.meta) {
+        for (user_key, user_id) in identity_map_from_meta(&ast.meta) {
             if let Some(slot) = identities.get_mut(&user_key) {
                 *slot = user_id;
             }