Skip to content

feat: wip v3 improvements #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kantorcodes wants to merge 3 commits into from
Closed

feat: wip v3 improvements #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
041e3ea
feat: wip v3 improvements
kantorcodes Sep 21, 2025
7cd8b5b
feat: wip
kantorcodes Sep 22, 2025
5230c25
Merge branch 'main' of https://github.com/hashgraph-online/desktop in…
kantorcodes Sep 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 32 additions & 11 deletions __tests__/components/chat/MessageList.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,20 @@ import '@testing-library/jest-dom';
import MessageList from '../../../src/renderer/components/chat/MessageList';
import type { Message } from '../../../src/renderer/stores/agentStore';

jest.mock('framer-motion', () => {
const React = require('react');
return {
motion: {
div: React.forwardRef(
(
props: React.HTMLAttributes<HTMLDivElement>,
ref: React.ForwardedRef<HTMLDivElement>
) => React.createElement('div', { ...props, ref })
),
},
};
});

jest.mock('../../../src/renderer/components/chat/MessageBubble', () => {
return function MockMessageBubble({ message }: { message: Message }) {
return <div data-testid={`message-${message.id}`}>{message.content}</div>;
Expand All @@ -20,6 +34,7 @@ Object.defineProperty(Element.prototype, 'scrollIntoView', {
writable: true,
});


describe('MessageList', () => {
const mockMessages: Message[] = [
{
Expand All @@ -39,9 +54,7 @@ describe('MessageList', () => {
it('should render empty state when no messages', () => {
render(<MessageList messages={[]} />);

expect(
screen.getByText('Welcome to Conversational Agent')
).toBeInTheDocument();
expect(screen.getByText('Welcome to HOL Desktop')).toBeInTheDocument();
expect(
screen.getByText(/I can help you with Hedera Hashgraph operations/)
).toBeInTheDocument();
Expand Down Expand Up @@ -77,21 +90,19 @@ describe('MessageList', () => {
it('should show loading indicator when isLoading is true', () => {
render(<MessageList messages={mockMessages} isLoading={true} />);

expect(screen.getByText('Agent is thinking...')).toBeInTheDocument();
expect(screen.getByText('Assistant is thinking...')).toBeInTheDocument();
});

it('should not show loading indicator when isLoading is false', () => {
render(<MessageList messages={mockMessages} isLoading={false} />);

expect(screen.queryByText('Agent is thinking...')).not.toBeInTheDocument();
expect(screen.queryByText('Assistant is thinking...')).not.toBeInTheDocument();
});

it('should not show empty state when messages are present', () => {
render(<MessageList messages={mockMessages} />);

expect(
screen.queryByText('Welcome to Conversational Agent')
).not.toBeInTheDocument();
expect(screen.queryByText('Welcome to HOL Desktop')).not.toBeInTheDocument();
});

it('should render messages in correct order', () => {
Expand Down Expand Up @@ -131,8 +142,18 @@ describe('MessageList', () => {
render(<MessageList messages={singleMessage} />);

expect(screen.getByTestId('message-msg-1')).toBeInTheDocument();
expect(
screen.queryByText('Welcome to Conversational Agent')
).not.toBeInTheDocument();
expect(screen.queryByText('Welcome to HOL Desktop')).not.toBeInTheDocument();
});

it('should render custom empty state when provided', () => {
render(
<MessageList
messages={[]}
emptyState={<div data-testid='custom-empty'>Moonscape Copilot is ready</div>}
/>
);

expect(screen.getByTestId('custom-empty')).toBeInTheDocument();
expect(screen.queryByText('Welcome to HOL Desktop')).not.toBeInTheDocument();
});
});
176 changes: 176 additions & 0 deletions __tests__/components/shell/browser-assistant-panel.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,176 @@
import React from 'react';
import { render, screen } from '@testing-library/react';
import '@testing-library/jest-dom';
import BrowserAssistantPanel from '../../../src/renderer/components/shell/BrowserAssistantPanel';

const agentStoreState: Record<string, unknown> = {};
const configStoreState: Record<string, unknown> = {};
const notificationStoreState: Record<string, unknown> = {};

jest.mock('framer-motion', () => {
const React = require('react');
const create = (element: string) =>
React.forwardRef((props: Record<string, unknown>, ref: React.Ref<HTMLElement>) => {
const {
whileHover,
whileTap,
initial,
animate,
exit,
transition,
layout,
variants,
addEventListener,
removeEventListener,
...rest
} = props;
return React.createElement(element, { ...rest, ref }, rest.children);
});
return {
motion: {
div: create('div'),
button: create('button'),
span: create('span'),
},
AnimatePresence: ({ children }: { children: React.ReactNode }) => <>{children}</>,
};
});

jest.mock('../../../src/renderer/lib/utils', () => ({
cn: (...classes: unknown[]) => classes.filter(Boolean).join(' '),
}));

jest.mock('../../../src/renderer/components/chat/headers/AnimatedSuggestions', () => ({
__esModule: true,
default: ({ onSelect }: { onSelect: (text: string) => void }) => (
<button type='button' data-testid='animated-suggestions' onClick={() => onSelect('')} />
),
}));

jest.mock('framer-motion', () => {
const React = require('react');
return {
motion: {
div: React.forwardRef(
(
props: React.HTMLAttributes<HTMLDivElement>,
ref: React.ForwardedRef<HTMLDivElement>
) => React.createElement('div', { ...props, ref })
),
},
};
});

jest.mock('react-router-dom', () => ({
useNavigate: () => jest.fn(),
}));

jest.mock('../../../src/renderer/hooks/useAgentInit', () => ({
useAgentInit: jest.fn(),
}));

const fileAttachmentsMock = jest.fn();

jest.mock('../../../src/renderer/hooks/useFileAttachments', () => ({
__esModule: true,
default: (...args: unknown[]) => fileAttachmentsMock(...args),
}));

jest.mock('../../../src/renderer/stores/agentStore', () => ({
useAgentStore: (selector: (state: Record<string, unknown>) => unknown) => selector(agentStoreState),
}));

jest.mock('../../../src/renderer/stores/configStore', () => ({
useConfigStore: (selector: (state: Record<string, unknown>) => unknown) => selector(configStoreState),
}));

jest.mock('../../../src/renderer/stores/notificationStore', () => ({
useNotificationStore: (selector: (state: Record<string, unknown>) => unknown) => selector(notificationStoreState),
}));

Object.defineProperty(window.HTMLElement.prototype, 'scrollIntoView', {
configurable: true,
value: jest.fn(),
writable: true,
});

const mediaQueryList = window.matchMedia('(prefers-reduced-motion: reduce)');
if (mediaQueryList) {
const prototype = Object.getPrototypeOf(mediaQueryList) as Record<string, unknown>;
if (typeof prototype.addEventListener !== 'function') {
prototype.addEventListener = jest.fn();
}
if (typeof prototype.removeEventListener !== 'function') {
prototype.removeEventListener = jest.fn();
}
}


describe('BrowserAssistantPanel', () => {
beforeEach(() => {
jest.clearAllMocks();

Object.keys(agentStoreState).forEach((key) => {
delete agentStoreState[key];
});
Object.assign(agentStoreState, {
isConnected: true,
status: 'connected',
connect: jest.fn(),
sendMessage: jest.fn(),
messages: [],
isTyping: false,
chatContext: { mode: 'personal' },
currentSession: { id: 'session-1', name: 'Session', mode: 'personal' },
createSession: jest.fn(),
loadSession: jest.fn(),
});

Object.keys(configStoreState).forEach((key) => {
delete configStoreState[key];
});
Object.assign(configStoreState, {
config: null,
isConfigured: true,
});

Object.keys(notificationStoreState).forEach((key) => {
delete notificationStoreState[key];
});
Object.assign(notificationStoreState, {
addNotification: jest.fn(),
});

fileAttachmentsMock.mockReturnValue({
files: [],
addFiles: jest.fn(),
removeFile: jest.fn(),
reset: jest.fn(),
fileError: null,
toBase64: jest.fn(),
});
});

it('renders browser-specific empty state copy', () => {
render(
<BrowserAssistantPanel
isOpen
sessionId='session-1'
hostLabel='hedera.kiloscribe.com'
currentUrl='https://hedera.kiloscribe.com'
pageTitle='Hedera Docs'
onSessionCreated={jest.fn()}
fetchPageContext={async () => null}
onClose={jest.fn()}
dock='right'
onDockChange={jest.fn()}
/>
);

expect(screen.getByText(/Moonscape Copilot/i)).toBeInTheDocument();
expect(screen.getByText(/Try asking/i)).toBeInTheDocument();
expect(
screen.queryByText(/I can help you with Hedera Hashgraph operations/)
).not.toBeInTheDocument();
});
});
Loading