Skip to content

refactor(kernel): fix None leak and migrate core utilities to /core#20

Closed
AkshajSinghal wants to merge 1 commit into
mainfrom
TruKernel
Closed

refactor(kernel): fix None leak and migrate core utilities to /core#20
AkshajSinghal wants to merge 1 commit into
mainfrom
TruKernel

Conversation

@AkshajSinghal
Copy link
Copy Markdown
Collaborator

  • Fix TruKernel _execute_entry to suppress printing of None return values.
  • Move database.py, model.py, and settings.py into trushell/core/ for better encapsulation.
  • Update all internal imports and test references to point to new trushell.core modules.
  • Begin migration of legacy command logic (pyfunny) into commands/joke.py.
  • Remove obsolete root-level utility files to reduce package clutter.

This refactor stabilizes the kernel's output handling and establishes the final internal directory structure before consolidating remaining legacy commands.

@AkshajSinghal
Copy link
Copy Markdown
Collaborator Author

Alright, this PR lands the core TruKernel engine. The manifest dispatch is working, OS fallback is live, and the None leak is fixed. I’ve also moved database, models, and settings into /core to clean up the root.

Heads up, I’m heading out. The migration isn’t 100% done.
These files are still sitting in the root but are effectively orphaned:

  • todocli.py (needs to merge into commands/tasks.py)
  • chronoterm/ (needs to merge into commands/chronoterm.py)
  • project.py (delete it, cli.py handles the REPL now)

If anyone wants to grab this branch and finish the cleanup, be my guest. Just make sure pytest -q passes before you push.
Don’t break the build. Cheers!

@AkshajSinghal AkshajSinghal changed the base branch from python-entry-point to main June 3, 2026 14:16
@AkshajSinghal
Copy link
Copy Markdown
Collaborator Author

Implemented in #21

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