Skip to content

add braindb to See also - #4

Closed
dimknaf wants to merge 1 commit into
Astro-Han:mainfrom
dimknaf:add-braindb
Closed

add braindb to See also#4
dimknaf wants to merge 1 commit into
Astro-Han:mainfrom
dimknaf:add-braindb

Conversation

@dimknaf

@dimknaf dimknaf commented May 25, 2026

Copy link
Copy Markdown

appends dimknaf/braindb to the See also comma-list at the bottom of the Inspired By section. one-character precision edit, matches the existing format exactly.

braindb is another implementation of Karpathy's wiki pattern — server-shaped (Postgres + pgvector + REST API + maintainer agent) where the existing See-also entries are CLI / MCP. fits the same audience.

@dimknaf

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Review Summary by Qodo

Add braindb to See also section

📝 Documentation

Grey Divider

Walkthroughs

Description
• Add braindb to See also section
• Includes link to dimknaf/braindb implementation
• Maintains existing comma-separated format
Diagram
flowchart LR
  A["See also list"] -- "append braindb link" --> B["Updated See also with three entries"]

Loading

File Changes

1. README.md 📝 Documentation +1/-1

Add braindb link to See also section

• Appends dimknaf/braindb to the See also comma-separated list
• Maintains consistent formatting with existing entries
• Adds another Karpathy wiki pattern implementation to related projects

README.md


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented May 25, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 49dfdb83-c723-4e27-bd8a-140ffc74cf1b

📥 Commits

Reviewing files that changed from the base of the PR and between 9e8c4f4 and 48140b1.

📒 Files selected for processing (1)
  • README.md
📜 Recent review details
🔇 Additional comments (1)
README.md (1)

128-128: LGTM!


📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated the "Inspired By" section to include an additional related project reference.

Walkthrough

This PR updates the README.md file to add a new reference to the dimknaf/braindb repository in the "Inspired By" section. The change introduces an additional "See also" link alongside the existing references to lucasastorian/llmwiki and atomicmemory/llm-wiki-compiler, expanding the acknowledgment of related projects.

Changes

README Documentation Update

Layer / File(s) Summary
README Inspired By Section Update
README.md
"Inspired By" section updated to add dimknaf/braindb as an additional "See also" reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A whisker-twitch of gratitude,
One link becomes now two plus more,
The braindb joins the crew—
A humble nod from readme's door!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'add braindb to See also' directly and clearly describes the main change—adding a new reference to the README's See also section.
Description check ✅ Passed The description is directly related to the changeset, explaining what is being added (dimknaf/braindb), where it's being added (See also list), why it's relevant, and how it maintains consistency with existing format.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README.md to include a link to the braindb repository in the 'See also' section. The reviewer suggested improving the readability and maintainability of this section by converting the single-line list into an alphabetized bulleted list.

Comment thread README.md
Unofficial community implementation of the workflow from [Karpathy's LLM Wiki idea](https://gist.github.com/karpathy/442a6bf555914893e9891c11519de94f). The value here is the reusable workflow, prompt structure, and battle-tested knowledge-compilation rules.

See also: [lucasastorian/llmwiki](https://github.com/lucasastorian/llmwiki), [atomicmemory/llm-wiki-compiler](https://github.com/atomicmemory/llm-wiki-compiler).
See also: [lucasastorian/llmwiki](https://github.com/lucasastorian/llmwiki), [atomicmemory/llm-wiki-compiler](https://github.com/atomicmemory/llm-wiki-compiler), [dimknaf/braindb](https://github.com/dimknaf/braindb).

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

As the list of related projects grows, the single-line format becomes difficult to read and maintain due to excessive line length. Converting this to an alphabetized bulleted list improves readability and follows common Markdown practices for maintainability.

Suggested change
See also: [lucasastorian/llmwiki](https://github.com/lucasastorian/llmwiki), [atomicmemory/llm-wiki-compiler](https://github.com/atomicmemory/llm-wiki-compiler), [dimknaf/braindb](https://github.com/dimknaf/braindb).
See also:
- [atomicmemory/llm-wiki-compiler](https://github.com/atomicmemory/llm-wiki-compiler)
- [dimknaf/braindb](https://github.com/dimknaf/braindb)
- [lucasastorian/llmwiki](https://github.com/lucasastorian/llmwiki)

@Astro-Han

Copy link
Copy Markdown
Owner

Thanks for the PR, and braindb looks like a solid take on the pattern. Going to pass though: See also is a short curated list I maintain myself, not something open to submissions. Happy to revisit as the project grows.

@Astro-Han Astro-Han closed this Jul 23, 2026
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.

2 participants