mirror of
https://github.com/hanzoai/opencode-openai-codex-auth.git
synced 2026-09-24 17:01:02 +00:00
## CODEX_MODE Implementation
Added configurable CODEX_MODE that replaces OpenCode system prompts with
a Codex-OpenCode bridge prompt for better Codex CLI parity.
**Key Features:**
- Enabled by default for optimal Codex CLI compatibility
- Configurable via ~/.opencode/openai-codex-auth-config.json
- Environment variable override support (CODEX_MODE=0 or CODEX_MODE=1)
- Priority: env var > config file > default (true)
**Bridge Prompt:**
- ~450 tokens (~90% reduction vs full OpenCode prompt)
- Critical tool replacements (apply_patch → edit, update_plan → todowrite)
- Available tools list and verification checklist
- OpenCode working style guidelines
**Configuration:**
```json
{
"codexMode": true // default
}
```
## Library Reorganization
Restructured lib/ into semantic subfolders:
```
lib/
├── auth/ # OAuth authentication
│ ├── auth.ts
│ ├── browser.ts
│ └── server.ts
├── prompts/ # System prompts
│ ├── codex.ts
│ └── codex-opencode-bridge.ts
├── request/ # Request handling
│ ├── fetch-helpers.ts
│ ├── request-transformer.ts
│ └── response-handler.ts
├── config.ts # Plugin configuration
├── constants.ts
├── logger.ts
├── types.ts
└── oauth-success.html
```
## Files Changed
**New Files:**
- lib/config.ts - Plugin configuration loading
- lib/prompts/codex-opencode-bridge.ts - Bridge prompt
- test/plugin-config.test.ts - 12 new tests
**Modified:**
- lib/request/request-transformer.ts - CODEX_MODE parameter
- lib/request/fetch-helpers.ts - Pass codexMode through pipeline
- index.ts - Load config and determine codexMode
- lib/types.ts - Add PluginConfig type
- README.md - Document CODEX_MODE configuration
**Reorganized:**
- lib/*.ts → lib/{auth,prompts,request}/*.ts
- Updated all imports across codebase and tests
## Testing
- 123 total tests (added 12 new tests for config loading)
- All tests passing
- Comprehensive CODEX_MODE coverage:
- Config file loading (exists, missing, invalid)
- Priority order testing
- Environment variable override
- Bridge prompt vs tool remap behavior
## Documentation
Updated README.md:
- Added "Plugin Configuration" section
- Documented CODEX_MODE setting and priority
- Updated features list (123 tests, CODEX_MODE)
- Updated "How It Works" section
Co-authored-by: Hanzo Dev <[email protected]>
144 lines
4.7 KiB
TypeScript
144 lines
4.7 KiB
TypeScript
import { describe, it, expect } from 'vitest';
|
|
import {
|
|
createState,
|
|
parseAuthorizationInput,
|
|
decodeJWT,
|
|
createAuthorizationFlow,
|
|
CLIENT_ID,
|
|
AUTHORIZE_URL,
|
|
REDIRECT_URI,
|
|
SCOPE,
|
|
} from '../lib/auth/auth.js';
|
|
|
|
describe('Auth Module', () => {
|
|
describe('createState', () => {
|
|
it('should generate a random 32-character hex string', () => {
|
|
const state = createState();
|
|
expect(state).toMatch(/^[a-f0-9]{32}$/);
|
|
});
|
|
|
|
it('should generate unique states', () => {
|
|
const state1 = createState();
|
|
const state2 = createState();
|
|
expect(state1).not.toBe(state2);
|
|
});
|
|
});
|
|
|
|
describe('parseAuthorizationInput', () => {
|
|
it('should parse full OAuth callback URL', () => {
|
|
const input = 'http://localhost:1455/auth/callback?code=abc123&state=xyz789';
|
|
const result = parseAuthorizationInput(input);
|
|
expect(result).toEqual({ code: 'abc123', state: 'xyz789' });
|
|
});
|
|
|
|
it('should parse code#state format', () => {
|
|
const input = 'abc123#xyz789';
|
|
const result = parseAuthorizationInput(input);
|
|
expect(result).toEqual({ code: 'abc123', state: 'xyz789' });
|
|
});
|
|
|
|
it('should parse query string format', () => {
|
|
const input = 'code=abc123&state=xyz789';
|
|
const result = parseAuthorizationInput(input);
|
|
expect(result).toEqual({ code: 'abc123', state: 'xyz789' });
|
|
});
|
|
|
|
it('should parse code only', () => {
|
|
const input = 'abc123';
|
|
const result = parseAuthorizationInput(input);
|
|
expect(result).toEqual({ code: 'abc123' });
|
|
});
|
|
|
|
it('should return empty object for empty input', () => {
|
|
const result = parseAuthorizationInput('');
|
|
expect(result).toEqual({});
|
|
});
|
|
|
|
it('should handle whitespace', () => {
|
|
const result = parseAuthorizationInput(' ');
|
|
expect(result).toEqual({});
|
|
});
|
|
});
|
|
|
|
describe('decodeJWT', () => {
|
|
it('should decode valid JWT token', () => {
|
|
// Create a simple JWT token: header.payload.signature
|
|
const header = Buffer.from(JSON.stringify({ alg: 'HS256', typ: 'JWT' })).toString('base64');
|
|
const payload = Buffer.from(JSON.stringify({ sub: '1234567890', name: 'Test User' })).toString('base64');
|
|
const signature = 'fake-signature';
|
|
const token = `${header}.${payload}.${signature}`;
|
|
|
|
const decoded = decodeJWT(token);
|
|
expect(decoded).toEqual({ sub: '1234567890', name: 'Test User' });
|
|
});
|
|
|
|
it('should decode JWT with ChatGPT account info', () => {
|
|
const payload = Buffer.from(JSON.stringify({
|
|
'https://api.openai.com/auth': {
|
|
chatgpt_account_id: 'account-123',
|
|
},
|
|
})).toString('base64');
|
|
const token = `header.${payload}.signature`;
|
|
|
|
const decoded = decodeJWT(token);
|
|
expect(decoded?.['https://api.openai.com/auth']?.chatgpt_account_id).toBe('account-123');
|
|
});
|
|
|
|
it('should return null for invalid JWT', () => {
|
|
const result = decodeJWT('invalid-token');
|
|
expect(result).toBeNull();
|
|
});
|
|
|
|
it('should return null for malformed JWT', () => {
|
|
const result = decodeJWT('header.payload');
|
|
expect(result).toBeNull();
|
|
});
|
|
|
|
it('should return null for non-JSON payload', () => {
|
|
const token = 'header.not-json.signature';
|
|
const result = decodeJWT(token);
|
|
expect(result).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('createAuthorizationFlow', () => {
|
|
it('should create authorization flow with PKCE', async () => {
|
|
const flow = await createAuthorizationFlow();
|
|
|
|
expect(flow).toHaveProperty('pkce');
|
|
expect(flow).toHaveProperty('state');
|
|
expect(flow).toHaveProperty('url');
|
|
|
|
expect(flow.pkce).toHaveProperty('challenge');
|
|
expect(flow.pkce).toHaveProperty('verifier');
|
|
expect(flow.state).toMatch(/^[a-f0-9]{32}$/);
|
|
});
|
|
|
|
it('should generate URL with correct parameters', async () => {
|
|
const flow = await createAuthorizationFlow();
|
|
const url = new URL(flow.url);
|
|
|
|
expect(url.origin + url.pathname).toBe(AUTHORIZE_URL);
|
|
expect(url.searchParams.get('response_type')).toBe('code');
|
|
expect(url.searchParams.get('client_id')).toBe(CLIENT_ID);
|
|
expect(url.searchParams.get('redirect_uri')).toBe(REDIRECT_URI);
|
|
expect(url.searchParams.get('scope')).toBe(SCOPE);
|
|
expect(url.searchParams.get('code_challenge_method')).toBe('S256');
|
|
expect(url.searchParams.get('code_challenge')).toBe(flow.pkce.challenge);
|
|
expect(url.searchParams.get('state')).toBe(flow.state);
|
|
expect(url.searchParams.get('id_token_add_organizations')).toBe('true');
|
|
expect(url.searchParams.get('codex_cli_simplified_flow')).toBe('true');
|
|
expect(url.searchParams.get('originator')).toBe('codex_cli_rs');
|
|
});
|
|
|
|
it('should generate unique flows', async () => {
|
|
const flow1 = await createAuthorizationFlow();
|
|
const flow2 = await createAuthorizationFlow();
|
|
|
|
expect(flow1.state).not.toBe(flow2.state);
|
|
expect(flow1.pkce.verifier).not.toBe(flow2.pkce.verifier);
|
|
expect(flow1.url).not.toBe(flow2.url);
|
|
});
|
|
});
|
|
});
|