Skip to content

refactor: chats method shebangs; drop chats index (D14) - #11

Merged
eSlider merged 1 commit into
mainfrom
feat/chats-d14
Aug 13, 2026
Merged

eSlider merged 1 commit into
mainfrom
feat/chats-d14

Conversation

@eSlider

@eSlider eSlider commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • bin/chats/{sync,import,facts,apply}.go are shebang mains (exclusive build tags). Shared code moved to internal/chats.
  • Removed bin/chats/main.go and chats index (brain write belonged on bin/brain/index.go). Facts still write JSON/markdown under var/chats/facts/.
  • Layout unittest gates the paths. LinkedIn/import fixtures stay synthetic (Alice/Bob/Charlie).

Gitea: https://git.produktor.io/eSlider/2dph/issues/13

Test plan

  • go test ./internal/chats ./...
  • uv run python -m unittest bin.tools.test_bin_layout
  • GitHub Actions green

Parsers and commands live in internal/chats. bin/chats/{sync,import,facts,apply}.go
are tagged shebang mains. Brain ingest is not a chats command.
@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 5d4b342 into main Aug 13, 2026
2 checks passed
@eSlider
eSlider deleted the feat/chats-d14 branch August 13, 2026 16:31
eSlider added a commit that referenced this pull request Aug 13, 2026
Parsers and commands live in internal/chats. bin/chats/{sync,import,facts,apply}.go
are tagged shebang mains. Brain ingest is not a chats command.
eSlider added a commit that referenced this pull request Aug 24, 2026
Parsers and commands live in internal/chats. bin/chats/{sync,import,facts,apply}.go
are tagged shebang mains. Brain ingest is not a chats command.
eSlider added a commit that referenced this pull request Sep 15, 2026
Parsers and commands live in internal/chats. bin/chats/{sync,import,facts,apply}.go
are tagged shebang mains. Brain ingest is not a chats command.
eSlider pushed a commit that referenced this pull request Sep 15, 2026
…follow-up #11)' (#306) from fix/skills-bin-paths#3 into main
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