feat(coding-agent): /reload now re-renders entire scrollback
For extension development, reloading now rebuilds the full chat history so updated component rendering is visible immediately, not just in new messages. Also removes broken test for deleted chalk-logger.ts extension (npm dependency resolution is already covered by with-deps test).
This commit is contained in:
@@ -23,6 +23,7 @@
|
||||
|
||||
### Changed
|
||||
|
||||
- `/reload` now re-renders the entire scrollback so updated extension components are visible immediately
|
||||
- Skill, prompt template, and theme discovery now use settings and CLI path arrays instead of legacy filters ([#645](https://github.com/badlogic/pi-mono/issues/645))
|
||||
|
||||
### Fixed
|
||||
|
||||
@@ -3361,6 +3361,7 @@ export class InteractiveMode {
|
||||
if (runner) {
|
||||
this.setupExtensionShortcuts(runner);
|
||||
}
|
||||
this.rebuildChatFromMessages();
|
||||
restoreEditor();
|
||||
this.showLoadedResources({ extensionPaths: runner?.getExtensionPaths() ?? [], force: true });
|
||||
const modelsJsonError = this.session.modelRegistry.getError();
|
||||
|
||||
@@ -297,19 +297,6 @@ describe("extensions discovery", () => {
|
||||
expect(result.extensions[0].path).toContain("my-ext.ts");
|
||||
});
|
||||
|
||||
it("resolves 3rd party npm dependencies (chalk)", async () => {
|
||||
// Load the real chalk-logger extension from examples
|
||||
const chalkLoggerPath = path.resolve(__dirname, "../examples/extensions/chalk-logger.ts");
|
||||
|
||||
const result = await discoverAndLoadExtensions([chalkLoggerPath], tempDir, tempDir);
|
||||
|
||||
expect(result.errors).toHaveLength(0);
|
||||
expect(result.extensions).toHaveLength(1);
|
||||
expect(result.extensions[0].path).toContain("chalk-logger.ts");
|
||||
// The extension registers event handlers, not commands/tools
|
||||
expect(result.extensions[0].handlers.size).toBeGreaterThan(0);
|
||||
});
|
||||
|
||||
it("resolves dependencies from extension's own node_modules", async () => {
|
||||
// Load extension that has its own package.json and node_modules with 'ms' package
|
||||
const extPath = path.resolve(__dirname, "../examples/extensions/with-deps");
|
||||
|
||||
Reference in New Issue
Block a user