fix(nats): align with movies-diary — Limits retention, single wildcard subject, filter_subject on consumer, prefixed publish

crates/adapters/nats/src/lib.rs

@@ -5,29 +5,16 @@ use event_transport::{MessageSource, RawMessage, Transport};
 use futures::stream::BoxStream;
 use std::sync::Arc;
 
-// Stream name and subjects used by both publisher and consumer.
 const STREAM_NAME: &str = "THOUGHTS_EVENTS";
-// Explicit prefixes instead of ">" — NATS WorkQueue retention disallows
-// the catch-all ">" wildcard without also setting no_ack = true.
-const STREAM_SUBJECTS: &[&str] = &[
-    "thoughts.>",
-    "likes.>",
-    "boosts.>",
-    "follows.>",
-    "users.>",
-    "federation.>",
-];
+const STREAM_SUBJECT: &str = "thoughts-events.>";
 const CONSUMER_NAME: &str = "worker";
-// Redelivery timeout: if a message is not acked within this time, NATS redelivers it.
-const ACK_WAIT_SECS: u64 = 30;
-// Maximum delivery attempts before the message goes to a dead-letter stream (if configured).
-const MAX_DELIVER: i64 = 5;
+const MAX_MESSAGES: i64 = 100_000;
 
 fn stream_config() -> StreamConfig {
     StreamConfig {
         name: STREAM_NAME.to_string(),
-        subjects: STREAM_SUBJECTS.iter().map(|s| s.to_string()).collect(),
-        retention: jetstream::stream::RetentionPolicy::WorkQueue,
+        subjects: vec![STREAM_SUBJECT.to_string()],
+        max_messages: MAX_MESSAGES,
         ..Default::default()
     }
 }
@@ -40,7 +27,7 @@ pub async fn ensure_stream(client: &async_nats::Client) -> Result<(), DomainErro
     // Falls back to create if the stream doesn't exist yet.
     match js.update_stream(stream_config()).await {
         Ok(_) => {
-            tracing::info!(subjects = ?STREAM_SUBJECTS, "JetStream stream updated");
+            tracing::info!(subject = STREAM_SUBJECT, "JetStream stream updated");
             Ok(())
         }
         Err(e) => {
@@ -70,8 +57,10 @@ impl NatsTransport {
 #[async_trait]
 impl Transport for NatsTransport {
     async fn publish_bytes(&self, subject: &str, bytes: &[u8]) -> Result<(), DomainError> {
+        // Prefix all subjects so they land inside the stream's subject filter.
+        let full_subject = format!("thoughts-events.{subject}");
         self.jetstream
-            .publish(subject.to_string(), bytes.to_vec().into())
+            .publish(full_subject, bytes.to_vec().into())
             .await
             .map_err(|e| DomainError::Internal(e.to_string()))?
             .await // wait for server ack — confirms message is durably stored
@@ -129,9 +118,7 @@ impl MessageSource for NatsMessageSource {
                     CONSUMER_NAME,
                     jetstream::consumer::pull::Config {
                         durable_name: Some(CONSUMER_NAME.to_string()),
-                        ack_policy: jetstream::consumer::AckPolicy::Explicit,
-                        ack_wait: std::time::Duration::from_secs(ACK_WAIT_SECS),
-                        max_deliver: MAX_DELIVER,
+                        filter_subject: STREAM_SUBJECT.to_string(),
                         ..Default::default()
                     },
                 )