Merge pull request #590 from AdaWorldAPI/claude/fix-589-mod-deprecated-allow

fix(bridges): guard deprecated-alias re-exports + ontology self-impls (#589 follow-up)

Author: AdaWorldAPI

crates/lance-graph-ogar/src/bridges/mod.rs

@@ -63,13 +63,32 @@ mod redmine_bridge;
 mod smb_bridge;
 mod woa_bridge;
 
-pub use medcare_bridge::{HealthcarePort, MedcareBridge};
-pub use odoo_bridge::{OdooBridge, OdooPort};
-pub use openproject_bridge::{OpenProjectBridge, OpenProjectPort};
-pub use redmine_bridge::{RedmineBridge, RedminePort};
-pub use smb_bridge::{SmbBridge, SmbPort};
+// Canonical surface — the `*Port` types and the `UnifiedBridge` harness
+// are the replacement and are NOT deprecated.
+pub use medcare_bridge::HealthcarePort;
+pub use odoo_bridge::OdooPort;
+pub use openproject_bridge::OpenProjectPort;
+pub use redmine_bridge::RedminePort;
+pub use smb_bridge::SmbPort;
 pub use unified::UnifiedBridge;
-pub use woa_bridge::{WoaBridge, WoaPort};
+pub use woa_bridge::WoaPort;
+
+// Deprecated per-port bridge aliases (2026-06-22) — pull the classid via
+// the corresponding PortSpec instead. The `#[allow(deprecated)]` here
+// silences the re-export site; the lint still fires at consumer
+// use-sites. See `docs/CONSUMER-BRIDGE-DEPRECATION.md` + AdaWorldAPI/OGAR#95.
+#[allow(deprecated)]
+pub use medcare_bridge::MedcareBridge;
+#[allow(deprecated)]
+pub use odoo_bridge::OdooBridge;
+#[allow(deprecated)]
+pub use openproject_bridge::OpenProjectBridge;
+#[allow(deprecated)]
+pub use redmine_bridge::RedmineBridge;
+#[allow(deprecated)]
+pub use smb_bridge::SmbBridge;
+#[allow(deprecated)]
+pub use woa_bridge::WoaBridge;
 
 // Compatibility shims for the pre-migration constants. `bridges`
 // previously re-exported `OPENPROJECT_CODEBOOK` / `REDMINE_CODEBOOK`

crates/lance-graph-ontology/src/bridges/ogit_bridge.rs

@@ -8,6 +8,11 @@
 //! A common pattern is to spin one `OgitBridge` per OGIT namespace that
 //! the consumer cares about, e.g. one for `Network`, another for `Auth`.
 
+// This module IS the deprecated `OgitBridge`'s implementation; its own
+// impl blocks reference the deprecated struct intentionally. Consumers in
+// other crates still get the warning. See docs/CONSUMER-BRIDGE-DEPRECATION.md.
+#![allow(deprecated)]
+
 use crate::bridge::{BridgeFromRegistry, NamespaceBridge};
 use crate::error::{Error, Result};
 use crate::namespace::NamespaceId;

crates/lance-graph-ontology/src/bridges/sharepoint_bridge.rs

@@ -22,6 +22,11 @@
 //! the scope-lock catches any cross-namespace leak before it reaches the
 //! orchestrator's storage schema.
 
+// This module IS the deprecated `SharePointBridge`'s implementation; its own
+// impl blocks reference the deprecated struct intentionally. Consumers in
+// other crates still get the warning. See docs/CONSUMER-BRIDGE-DEPRECATION.md.
+#![allow(deprecated)]
+
 use crate::bridge::{BridgeFromRegistry, NamespaceBridge};
 use crate::error::{Error, Result};
 use crate::namespace::NamespaceId;

crates/lance-graph-ontology/src/bridges/spear_bridge.rs

@@ -15,6 +15,11 @@
 //! catches any cross-namespace leak before it reaches Spear's columnar
 //! schema.
 
+// This module IS the deprecated `SpearBridge`'s implementation; its own
+// impl blocks reference the deprecated struct intentionally. Consumers in
+// other crates still get the warning. See docs/CONSUMER-BRIDGE-DEPRECATION.md.
+#![allow(deprecated)]
+
 use crate::bridge::{BridgeFromRegistry, NamespaceBridge};
 use crate::error::{Error, Result};
 use crate::namespace::NamespaceId;

crates/lance-graph-ontology/src/bridges/woa_bridge.rs

@@ -2,6 +2,11 @@
 //! namespace. Phase 6 of this session emits the corresponding TTL into
 //! `AdaWorldAPI/OGIT/NTO/WorkOrder/`.
 
+// This module IS the deprecated `WoaBridge`'s implementation; its own impl
+// blocks + `_compile_check` reference the deprecated struct intentionally.
+// Consumers in other crates still get the warning. See docs/CONSUMER-BRIDGE-DEPRECATION.md.
+#![allow(deprecated)]
+
 use crate::bridge::{BridgeError, BridgeFromRegistry, NamespaceBridge};
 use crate::error::{Error, Result};
 use crate::namespace::NamespaceId;