From 3995e62a33bb02807db6df486281c83a16b62022 Mon Sep 17 00:00:00 2001
From: Wez Furlong <wez@wezfurlong.org>
Date: Fri, 4 Apr 2025 14:40:19 -0700
Subject: [PATCH] message: disambiguate "metadata must be loaded first" errors

We have a couple of different places that might emit it,
let's annotate each one so that we know where it might
be coming from.
---
 crates/message/src/message.rs | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/crates/message/src/message.rs b/crates/message/src/message.rs
index 7a754567..225ac91a 100644
--- a/crates/message/src/message.rs
+++ b/crates/message/src/message.rs
@@ -363,7 +363,7 @@ impl Message {
     ) -> anyhow::Result<Option<Scheduling>> {
         let mut inner = self.msg_and_id.inner.lock();
         match &mut inner.metadata {
-            None => anyhow::bail!("metadata must be loaded first"),
+            None => anyhow::bail!("set_scheduling: metadata must be loaded first"),
             Some(meta) => {
                 meta.schedule = scheduling;
                 inner
@@ -733,7 +733,7 @@ impl Message {
     ) -> anyhow::Result<()> {
         let mut inner = self.msg_and_id.inner.lock();
         match &mut inner.metadata {
-            None => anyhow::bail!("metadata must be loaded first"),
+            None => anyhow::bail!("set_meta: metadata must be loaded first"),
             Some(meta) => {
                 let key = key.as_ref();
                 let value = value.into();
@@ -768,7 +768,7 @@ impl Message {
     pub fn get_meta_obj(&self) -> anyhow::Result<serde_json::Value> {
         let inner = self.msg_and_id.inner.lock();
         match &inner.metadata {
-            None => anyhow::bail!("metadata must be loaded first"),
+            None => anyhow::bail!("get_meta_obj: metadata must be loaded first"),
             Some(meta) => Ok(meta.meta.clone()),
         }
     }
@@ -779,7 +779,7 @@ impl Message {
     ) -> anyhow::Result<serde_json::Value> {
         let inner = self.msg_and_id.inner.lock();
         match &inner.metadata {
-            None => anyhow::bail!("metadata must be loaded first"),
+            None => anyhow::bail!("get_meta: metadata must be loaded first"),
             Some(meta) => match meta.meta.get(key) {
                 Some(value) => Ok(value.clone()),
                 None => Ok(serde_json::Value::Null),