Skip to content

Comments

Implement Git-mastery REPL#55

Open
jiaxinnns wants to merge 5 commits intogit-mastery:mainfrom
jiaxinnns:feat/repl
Open

Implement Git-mastery REPL#55
jiaxinnns wants to merge 5 commits intogit-mastery:mainfrom
jiaxinnns:feat/repl

Conversation

@jiaxinnns
Copy link

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements an interactive REPL (Read-Eval-Print Loop) for Git-Mastery, allowing users to execute Git-Mastery commands and shell commands in an interactive session. The REPL provides a convenient way to work with Git-Mastery without repeatedly typing the full command prefix. Additionally, it updates the module caching mechanism in the exercise loader to ensure fresh imports in the REPL context.

Changes:

  • Added a new interactive REPL command with support for Git-Mastery commands, shell command pass-through, and built-in commands like cd, exit, and help
  • Enhanced module cleanup in the exercise loader to prevent stale module caches in persistent REPL sessions
  • Integrated the new REPL command into the CLI command registry

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
app/commands/repl.py Implements the interactive REPL with command execution, shell pass-through, and help system
app/utils/gitmastery.py Adds module cache cleanup before and after exercise file execution to ensure fresh imports in REPL context
app/commands/init.py Exports the new repl command for use in the CLI
app/cli.py Registers the repl command in the CLI command list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

REPL for Git-Mastery

1 participant