Repository navigation
CLI output drops or crashes on bracketed values; duplicate deprecation line; insert_after_section blank line #1720
Description
Activity
Claude finished @phernandez's task in 16s —— View job
Triage
- Type: bug (three small defects found in the v0.24.0 acceptance test). I applied the
buglabel. - Component: CLI (items 1 and 2) and MCP tools (item 3,
edit_note). I did not apply thecloudlabel. Thebm cloud sync,bisyncandbisync-resetcommands in item 2 are in the CLI, but the bug is Typer's duplicate warning, not Cloud behavior. - Priority:
- Item 1 is medium. Bracketed values get silently dropped, and a value like
a[/b]craisesMarkupErrorand crashes the command. - Items 2 and 3 are low. Item 2 is a duplicate warning line. Item 3 is a stray blank line that splits a list.
- Item 1 is medium. Bracketed values get silently dropped, and a value like
- Complexity: simple to medium. Item 1 touches about 59 call sites, but the fix is the same everywhere: use
rich.markup.escapeormarkup=False, as feat(cli): deprecate cloud bisync and mirror sync; diagnostics env overrides #1711 did. Items 2 and 3 are each localized fixes. - Duplicates: I searched and found none. Related but closed: Deprecate bm cloud bisync and mirror sync; make push/pull the only supported cloud sync #1596 (the deprecation of these commands) and [FEATURE] Add insert_before_section and insert_after_section operations to edit_note #646 (the original
insert_after_sectionfeature).
Notes for whoever picks this up:
- Item 1 needs a regression test with a bracketed project name and a value containing a closing tag.
- Item 3 needs a test that inserting before a list item keeps the list contiguous.
- Per the repo's coverage rule, new code needs tests.
I have only triaged this. I made no code changes.
- Type: bug (three small defects found in the v0.24.0 acceptance test). I applied the
- added 5 commits that reference this issue
on Oct 10, 2026
Found during the v0.24.0 acceptance test.
console.print(f"[style]...{value}...[/style]")calls insrc/basic_memory/cliput user values (project names, paths, exception text) directly into Rich markup. Bracketed text is dropped (proj [x]prints asproj), a value with a closing tag such asa[/b]craisesMarkupErrorand crashes the command, and[bold]restyles the output. Escape the values withrich.markup.escape, or print them withmarkup=False, as feat(cli): deprecate cloud bisync and mirror sync; diagnostics env overrides #1711 did for the mirror notices.bm cloud sync,bisyncandbisync-resetprint Typer's ownDeprecationWarning: The command 'sync' is deprecated.line above our notice, so the warning appears twice. Keep the(deprecated)mark in help and drop the duplicate runtime line.edit_noteinsert_after_sectionleaves a blank line between the inserted content and the section's existing first line, which splits a list in two.