diff --git a/HumanCapital/ICompensationEstimationService.cs b/HumanCapital/ICompensationEstimationService.cs index dd25564..f66824d 100644 --- a/HumanCapital/ICompensationEstimationService.cs +++ b/HumanCapital/ICompensationEstimationService.cs @@ -8,6 +8,12 @@ namespace MoneyMirror.HumanCapital; /// before #24 (real wage-data retrieval + deterministic aggregation) is /// built; see #148 for the swap-out plan. /// +/// +/// #266: #24's real, BLS-grounded replacement already exists and is +/// registered in DI - see - but +/// nothing in the UI calls it yet. See that type's remarks for why the +/// swap hasn't happened. +/// public interface ICompensationEstimationService { /// @@ -15,5 +21,6 @@ public interface ICompensationEstimationService /// Task EstimateAsync( IReadOnlyList occupations, - CancellationToken cancellationToken = default); + CancellationToken cancellationToken = default + ); } diff --git a/HumanCapital/IMarketPotentialPipeline.cs b/HumanCapital/IMarketPotentialPipeline.cs index 1943498..16d9809 100644 --- a/HumanCapital/IMarketPotentialPipeline.cs +++ b/HumanCapital/IMarketPotentialPipeline.cs @@ -8,6 +8,19 @@ namespace MoneyMirror.HumanCapital; /// prose (#99). No AI participates in choosing the numbers - only in /// describing them afterward. /// +/// +/// #266: not currently called from any page - /market-potential and +/// the Dashboard still use the AI-guess-only +/// (#148), whose UI explicitly labels its output "AI estimate - not based +/// on live wage data". This isn't an oversight: this method needs a BLS +/// series ID per matched occupation, which was meant to come from O*NET +/// occupation data (#93/#94, HC4) - but eBay and O*NET were both dropped +/// from project scope (see #177) before that occupation-to-series mapping +/// was built. Wiring this in requires deciding how to get that mapping +/// without O*NET (a fixed lookup table for common titles? a second LLM +/// call to guess a series ID, no longer "no AI in choosing the number"?) +/// before this pipeline can replace the AI-guess path in the UI. +/// public interface IMarketPotentialPipeline { ///