mirror of
https://github.com/garrytan/gstack.git
synced 2026-05-06 05:35:46 +02:00
001ba59be0
Refactors checkTabAccess(tabId, clientId, isWrite) to use an options
object { isWrite?, ownOnly? }. Adds tabPolicy === 'own-only' support
in the server command dispatch — scoped tokens with this policy are
restricted to their own tabs for all commands, not just writes.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
245 lines
8.4 KiB
TypeScript
245 lines
8.4 KiB
TypeScript
/**
|
|
* Tab isolation tests — verify per-agent tab ownership in BrowserManager.
|
|
*
|
|
* These test the ownership Map and checkTabAccess() logic directly,
|
|
* without launching a browser (pure logic tests).
|
|
*/
|
|
|
|
import { describe, it, expect, beforeEach } from 'bun:test';
|
|
import { BrowserManager } from '../src/browser-manager';
|
|
|
|
// We test the ownership methods directly. BrowserManager can't call newTab()
|
|
// without a browser, so we test the ownership map + access checks via
|
|
// the public API that doesn't require Playwright.
|
|
|
|
describe('Tab Isolation', () => {
|
|
let bm: BrowserManager;
|
|
|
|
beforeEach(() => {
|
|
bm = new BrowserManager();
|
|
});
|
|
|
|
describe('getTabOwner', () => {
|
|
it('returns null for tabs with no owner', () => {
|
|
expect(bm.getTabOwner(1)).toBeNull();
|
|
expect(bm.getTabOwner(999)).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('checkTabAccess', () => {
|
|
it('root can always access any tab (read)', () => {
|
|
expect(bm.checkTabAccess(1, 'root', { isWrite: false })).toBe(true);
|
|
});
|
|
|
|
it('root can always access any tab (write)', () => {
|
|
expect(bm.checkTabAccess(1, 'root', { isWrite: true })).toBe(true);
|
|
});
|
|
|
|
it('any agent can read an unowned tab', () => {
|
|
expect(bm.checkTabAccess(1, 'agent-1', { isWrite: false })).toBe(true);
|
|
});
|
|
|
|
it('scoped agent cannot write to unowned tab', () => {
|
|
expect(bm.checkTabAccess(1, 'agent-1', { isWrite: true })).toBe(false);
|
|
});
|
|
|
|
it('scoped agent can read another agent tab', () => {
|
|
// Simulate ownership by using transferTab on a fake tab
|
|
// Since we can't create real tabs without a browser, test the access check
|
|
// with a known owner via the internal state
|
|
// We'll use transferTab which only checks pages map... let's test checkTabAccess directly
|
|
// checkTabAccess reads from tabOwnership map, which is empty here
|
|
expect(bm.checkTabAccess(1, 'agent-2', { isWrite: false })).toBe(true);
|
|
});
|
|
|
|
it('scoped agent cannot write to another agent tab', () => {
|
|
// With no ownership set, this is an unowned tab -> denied
|
|
expect(bm.checkTabAccess(1, 'agent-2', { isWrite: true })).toBe(false);
|
|
});
|
|
});
|
|
|
|
describe('transferTab', () => {
|
|
it('throws for non-existent tab', () => {
|
|
expect(() => bm.transferTab(999, 'agent-1')).toThrow('Tab 999 not found');
|
|
});
|
|
});
|
|
});
|
|
|
|
// Test the instruction block generator
|
|
import { generateInstructionBlock } from '../src/cli';
|
|
|
|
describe('generateInstructionBlock', () => {
|
|
it('generates a valid instruction block with setup key', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_test123',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('gsk_setup_test123');
|
|
expect(block).toContain('https://test.ngrok.dev/connect');
|
|
expect(block).toContain('STEP 1');
|
|
expect(block).toContain('STEP 2');
|
|
expect(block).toContain('STEP 3');
|
|
expect(block).toContain('COMMAND REFERENCE');
|
|
expect(block).toContain('read + write access');
|
|
expect(block).toContain('tabId');
|
|
expect(block).toContain('@ref');
|
|
expect(block).not.toContain('undefined');
|
|
});
|
|
|
|
it('uses localhost URL when no tunnel', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_local',
|
|
serverUrl: 'http://127.0.0.1:45678',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: 'in 24 hours',
|
|
});
|
|
|
|
expect(block).toContain('http://127.0.0.1:45678/connect');
|
|
});
|
|
|
|
it('shows admin scope description when admin included', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_admin',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write', 'admin', 'meta'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('admin access');
|
|
expect(block).toContain('execute JS');
|
|
expect(block).not.toContain('re-pair with --admin');
|
|
});
|
|
|
|
it('shows re-pair hint when admin not included', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_nonadmin',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('re-pair with --admin');
|
|
});
|
|
|
|
it('includes newtab as step 2 (agents must own their tab)', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_test',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('Create your own tab');
|
|
expect(block).toContain('"command": "newtab"');
|
|
});
|
|
|
|
it('includes error troubleshooting section', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_test',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('401');
|
|
expect(block).toContain('403');
|
|
expect(block).toContain('429');
|
|
});
|
|
|
|
it('teaches the snapshot→@ref pattern', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_snap',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
// Must explain the snapshot→@ref workflow
|
|
expect(block).toContain('snapshot');
|
|
expect(block).toContain('@e1');
|
|
expect(block).toContain('@e2');
|
|
expect(block).toContain("Always snapshot first");
|
|
expect(block).toContain("Don't guess selectors");
|
|
});
|
|
|
|
it('shows SERVER URL prominently', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_url',
|
|
serverUrl: 'https://my-tunnel.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('SERVER: https://my-tunnel.ngrok.dev');
|
|
});
|
|
|
|
it('includes newtab in COMMAND REFERENCE', () => {
|
|
const block = generateInstructionBlock({
|
|
setupKey: 'gsk_setup_ref',
|
|
serverUrl: 'https://test.ngrok.dev',
|
|
scopes: ['read', 'write'],
|
|
expiresAt: '2026-04-06T00:00:00Z',
|
|
});
|
|
|
|
expect(block).toContain('"command": "newtab"');
|
|
expect(block).toContain('"command": "goto"');
|
|
expect(block).toContain('"command": "snapshot"');
|
|
expect(block).toContain('"command": "click"');
|
|
expect(block).toContain('"command": "fill"');
|
|
});
|
|
});
|
|
|
|
// Test CLI source-level behavior (pair-agent headed mode, ngrok detection)
|
|
import * as fs from 'fs';
|
|
import * as path from 'path';
|
|
|
|
const CLI_SRC = fs.readFileSync(path.join(import.meta.dir, '../src/cli.ts'), 'utf-8');
|
|
|
|
describe('pair-agent CLI behavior', () => {
|
|
// Extract the pair-agent block: from "pair-agent" dispatch to "process.exit(0)"
|
|
const pairStart = CLI_SRC.indexOf("command === 'pair-agent'");
|
|
const pairEnd = CLI_SRC.indexOf('process.exit(0)', pairStart);
|
|
const pairBlock = CLI_SRC.slice(pairStart, pairEnd);
|
|
|
|
it('auto-switches to headed mode unless --headless', () => {
|
|
expect(pairBlock).toContain("state.mode !== 'headed'");
|
|
expect(pairBlock).toContain("--headless");
|
|
expect(pairBlock).toContain("connect");
|
|
});
|
|
|
|
it('uses process.execPath for binary path (not argv[1] which is virtual in compiled)', () => {
|
|
expect(pairBlock).toContain('process.execPath');
|
|
// browseBin should be set to execPath, not argv[1]
|
|
expect(pairBlock).toContain('const browseBin = process.execPath');
|
|
});
|
|
|
|
it('isNgrokAvailable checks gstack env, NGROK_AUTHTOKEN, and native config', () => {
|
|
const ngrokBlock = CLI_SRC.slice(
|
|
CLI_SRC.indexOf('function isNgrokAvailable'),
|
|
CLI_SRC.indexOf('// ─── Pair-Agent DX')
|
|
);
|
|
// Three sources checked (paths are in path.join() calls, check the string literals)
|
|
expect(ngrokBlock).toContain("'ngrok.env'");
|
|
expect(ngrokBlock).toContain('NGROK_AUTHTOKEN');
|
|
expect(ngrokBlock).toContain("'ngrok.yml'");
|
|
// Checks macOS, Linux XDG, and legacy paths
|
|
expect(ngrokBlock).toContain("'Application Support'");
|
|
expect(ngrokBlock).toContain("'.config'");
|
|
expect(ngrokBlock).toContain("'.ngrok2'");
|
|
});
|
|
|
|
it('calls POST /tunnel/start when ngrok is available (not restart)', () => {
|
|
const handleBlock = CLI_SRC.slice(
|
|
CLI_SRC.indexOf('async function handlePairAgent'),
|
|
CLI_SRC.indexOf('function main()')
|
|
);
|
|
expect(handleBlock).toContain('/tunnel/start');
|
|
// Must NOT contain server restart logic
|
|
expect(handleBlock).not.toContain('Bun.spawn([\'bun\', \'run\'');
|
|
expect(handleBlock).not.toContain('BROWSE_TUNNEL');
|
|
});
|
|
});
|