Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🚀 Preview DeploymentYour preview deployment is ready! 🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-449 This preview will update automatically when you push new commits. |
|
|
||
| ### Last Will | ||
|
|
||
| Added in: v4.3.0 (inferred from version comparison, needs verification) |
There was a problem hiding this comment.
Do you think it would be possible to have all our "Added in: version" text be in a smaller/different font? And/or maybe right aligned, so it is little more subtle.
There was a problem hiding this comment.
Yes absolutely. I'll not this as part of the follow up tasks. Will need to use some sort of custom mdx component to do it but absolutely possible.
There was a problem hiding this comment.
I'm merging in a number of the "(inferred from version..." notes that the AI left behind so that I'm forced to go back through these files and ensure everything is cleaned up (links is a big one). So i'll make stylistic updates part of that pass too.
🚀 Preview DeploymentYour preview deployment is ready! 🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-449 This preview will update automatically when you push new commits. |
MQTT Section Migration
Summary
Migration of MQTT documentation from
versioned_docs/into the new v4 consolidated reference structure.Files Created
reference_versioned_docs/version-v4/mqtt/overview.mdreference_versioned_docs/version-v4/mqtt/configuration.mdSource Files Used
mqtt/overview.mdversioned_docs/version-4.7/developers/real-time.md(primary — MQTT section, Topics, Ordering, Events, Feature Support Matrix)release-notes/v4-tucker/4.2.0.md(confirmed MQTT introduction, QoS 0/1, durable sessions)release-notes/v4-tucker/4.3.0.md(confirmed mTLS, single-level wildcards, retain handling)release-notes/v4-tucker/4.5.0.md(confirmed improved message delivery)mqtt/configuration.mdversioned_docs/version-4.7/deployments/configuration.md(mqtt section, lines 1377–1446)versioned_docs/version-4.7/developers/real-time.md(configuration YAML example)release-notes/v4-tucker/4.2.0.md(confirmed mqtt config section added)release-notes/v4-tucker/4.3.0.md(confirmed mTLS config options added)Version Annotations Added
High Confidence (Confirmed via release notes)
+): Added in v4.3.0Needs Verification
mqtt.network.mtls.certificateVerification: Present in v4.7 config docs but exact introduction version not confirmed in release notesLink Placeholders Created
See
migration-context/link-placeholders/mqtt-link-placeholders.mdfor complete list.Summary:
security/overview.mddatabase/schema.mdhttp/tls.mdrest/overview.mdconfiguration/overview.mdImages/Assets Noted
None — MQTT docs contain no images.
Conflicts & Questions for Human Review
Content Notes
The migration map noted "Port change v4.5.0 (9925 → 9933)" for MQTT configuration. This port change was for the replication port, not the MQTT port. MQTT ports (1883/8883) have not changed across v4. The annotation in the configuration.md source comment has been preserved for awareness but not surfaced as a version annotation in the content.
The v4.7 source says: "Harper currently only supports trailing multi-level wildcards (no single-level wildcards with
*)" — but the feature matrix directly below in the same file marks single-level wildcards as ✅, and the 4.3.0 release notes confirm single-level wildcards were added. The body text appears to be outdated/stale. The generated docs use the feature matrix as authoritative. Human reviewer should confirm single-level wildcards work as expected.CRDT was noted in the migration map as a v4.3.0 MQTT feature. CRDT is a general data update mechanism surfaced via REST PATCH and Resource API — not MQTT-specific — so it is not included in the MQTT overview. If there's MQTT-specific CRDT behavior, please flag for addition.
Uncertainties
mqtt.network.mtls.certificateVerificationdefaults and options: sourced from the v4.7 config docs. The HTTP mTLS config has a more detailed CRL+OCSP description; MQTT's config docs only mention OCSP. This may reflect a genuine difference (MQTT only supports OCSP, not CRL). Human reviewer should verify.Migration Map Status
Updated status for
overview.mdandconfiguration.mdto "In Progress"Checklist for Human Reviewer
+) support — body text in source appears stale vs feature matrixcertificateVerificationonly supports OCSP (not CRL) unlike HTTP mTLS🤖 Generated with Claude Code