Discuss full decoupling from young-stock-cli and independent portfolio memory

[AdvancingTitans]

Context

stock-analysis previously shared runtime assumptions with young-stock-cli, including dependency installation, imported market-data core behavior, and portfolio-memory/profile expectations. This can cause cache/profile pollution and makes the skill harder for arbitrary agents to integrate independently.

Proposed direction

• Make stock-analysis fully runnable without users installing young-stock-cli.
• Keep the proven market data source order and investment-framework design inside stock-analysis.
• Move default cache/profile state into stock-analysis namespaces:
  • cache: ~/.cache/stock-analysis
  • portfolio memory: ~/.stock_analysis/profile.json or STOCK_ANALYSIS_PROFILE
• Never read or write young-stock-cli profile paths by default.
• For holdings-related requests, read the local stock-analysis memory first; if no complete memory exists, ask the user once for the missing portfolio fields.
• Keep ordinary market recaps free of portfolio sections unless holdings are explicitly requested.

Discussion points

• Whether the vendored data-source core should be accepted as the first clean break, then split into smaller modules later.
• Whether CLI should eventually support an interactive holdings-input command, or keep that interaction at the agent-skill layer.
• Whether any compatibility import/migration from old profile paths should exist as an explicit one-time tool, rather than default behavior.