Skip to content

feat: markdown import splits H2 leafs in Go - #26

Merged
eSlider merged 1 commit into
mainfrom
feat/markdown-import-go
Aug 13, 2026
Merged

eSlider merged 1 commit into
mainfrom
feat/markdown-import-go

Conversation

@eSlider

@eSlider eSlider commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • bin/markdown/import.go splits markdown on H2 in Go (internal/mdleaves). No ExecFile to Python.
  • Python bin/md/import stays as a fallback. Conversion still does not write Ladybug.
  • Adapter test: bin/kb/index --dry-run on a fixture corpus does not create kb.lbug.

Gitea: https://git.produktor.io/eSlider/2dph/issues/14 (slice 2), https://git.produktor.io/eSlider/2dph/issues/7

Test plan

  • go test ./internal/mdleaves
  • uv run python -m unittest discover -s bin/tools -t .
  • ./bin/markdown/import.go --files … --json on a fixture
  • CI green

Drop the Python ExecFile wrapper. Conversion still does not write
Ladybug; add a dry-run fixture check for the index adapter split.
@cursor

cursor Bot commented Aug 13, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@eSlider
eSlider merged commit b441af5 into main Aug 13, 2026
2 checks passed
@eSlider
eSlider deleted the feat/markdown-import-go branch August 13, 2026 22:27
eSlider added a commit that referenced this pull request Aug 13, 2026
Drop the Python ExecFile wrapper. Conversion still does not write
Ladybug; add a dry-run fixture check for the index adapter split.
eSlider added a commit that referenced this pull request Aug 24, 2026
Drop the Python ExecFile wrapper. Conversion still does not write
Ladybug; add a dry-run fixture check for the index adapter split.
eSlider added a commit that referenced this pull request Sep 15, 2026
Drop the Python ExecFile wrapper. Conversion still does not write
Ladybug; add a dry-run fixture check for the index adapter split.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant