You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
mm index <path> (packages/memtomem/src/memtomem/cli/indexing.py:107 and :194) calls comp.index_engine.index_path() — the non-stream path. So even after PR #653 added per-chunk SSE progress, direct CLI users see exactly the same thing they did before: nothing until the run finishes, then a single summary block.
The wizard already uses index_path_stream (init_cmd.py:_seed_with_progress) and gets per-file progress. Folding mm index into the same pattern would unify the CLI feedback story.
Proposal
Switch mm index to consume index_path_stream and render a click.progressbar matching the wizard's shape. Likely overlap with #655 (wizard chunk display) — both surfaces share the same click.progressbar constraints, so it's natural to either:
Context
mm index <path>(packages/memtomem/src/memtomem/cli/indexing.py:107and:194) callscomp.index_engine.index_path()— the non-stream path. So even after PR #653 added per-chunk SSE progress, direct CLI users see exactly the same thing they did before: nothing until the run finishes, then a single summary block.The wizard already uses
index_path_stream(init_cmd.py:_seed_with_progress) and gets per-file progress. Foldingmm indexinto the same pattern would unify the CLI feedback story.Proposal
Switch
mm indexto consumeindex_path_streamand render aclick.progressbarmatching the wizard's shape. Likely overlap with #655 (wizard chunk display) — both surfaces share the sameclick.progressbarconstraints, so it's natural to either:item_show_funchere.Acceptance criteria
mm index <large-dir>shows live per-file progress (and per-chunk if feat(cli): show chunk progress in mm init wizard's _seed_with_progress #655 lands first).Ctrl-Cmid-run still prints the same resume hint as today.--forceand--namespaceflags preserved.index_pathAPI doesn't need to be removed — internal callers (tests, MCPmem_index) may still want it.Files likely touched
packages/memtomem/src/memtomem/cli/indexing.py(the twoindex_pathcall sites)_run_indexing_with_barhelper between this file andinit_cmd.py:_seed_with_progress(depends on feat(cli): show chunk progress in mm init wizard's _seed_with_progress #655's outcome).References