Skip to content

Commit 320b06b

Browse files
fix(frontend): add New Session CTA to assistant workspace sidebar
- Include New Session primary CTA in assistant workspace view - Update test expectations for assistant workspace navigation model
1 parent a78e54a commit 320b06b

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

frontend/src/components/navigation/moreDrawerItems.test.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -139,12 +139,15 @@ describe('buildNavModel', () => {
139139
expect(model.primary[1].variant).toBe('secondary')
140140
})
141141

142-
it('returns assistant primary CTA for assistant workspace', () => {
142+
it('returns new-session and assistant primary CTAs for assistant workspace', () => {
143143
const model = buildNavModel('/repos/5/assistant')
144-
expect(model.primary).toHaveLength(1)
145-
expect(model.primary[0].key).toBe('assistant')
146-
expect(model.primary[0].to).toBe('/repos/5/assistant')
144+
expect(model.primary).toHaveLength(2)
145+
expect(model.primary[0].key).toBe('new-session')
146+
expect(model.primary[0].onSelect).toBe('new-session')
147147
expect(model.primary[0].variant).toBe('primary')
148+
expect(model.primary[1].key).toBe('assistant')
149+
expect(model.primary[1].to).toBe('/repos/5/assistant')
150+
expect(model.primary[1].variant).toBe('secondary')
148151
})
149152

150153
it('returns new-schedule primary CTA for schedules routes', () => {

frontend/src/components/navigation/moreDrawerItems.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,8 @@ export function buildNavModel(pathname: string, options: BuildNavOptions = {}):
121121

122122
return {
123123
primary: [
124-
getAssistantNavItem(pathname, 'primary'),
124+
{ key: 'new-session', label: 'New Session', icon: SquarePlus, onSelect: 'new-session', variant: 'primary' },
125+
getAssistantNavItem(pathname, 'secondary'),
125126
],
126127
items,
127128
}

0 commit comments

Comments
 (0)