docs(mock-attestation): note mock expiry only caps (vs Dstack full ownership); defer full alignment to #1639

Author: barakeinav1

crates/mpc-attestation/src/attestation.rs

@@ -151,6 +151,12 @@ impl MockAttestation {
     /// [`MockAttestation::WithConstraints`] carrying the cap, so that once stored
     /// it eventually expires and becomes eligible for cleanup.
     /// [`MockAttestation::Invalid`] is returned unchanged — it never reaches acceptance.
+    ///
+    // TODO(#1639): Dstack fully owns the expiry (the submission carries none, so
+    // the contract sets it outright); here a mock's submitted expiry is only capped.
+    // Full alignment — override the value, or split the submitted vs stored mock
+    // types (`ValidatedMockAttestation`) — is deferred; revisit alongside #1639
+    // (certificate-derived expiry).
     pub fn with_expiry(self, expiry_timestamp_seconds: u64) -> Self {
         match self {
             MockAttestation::Valid => MockAttestation::WithConstraints {