default github mode to be on.

bhouston · bhouston · commit 76bf76c47e74 · 2025-03-12T07:06:30.000-04:00
diff --git a/packages/agent/src/core/toolAgent.test.ts b/packages/agent/src/core/toolAgent.test.ts
@@ -14,7 +14,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 
 // Mock tool for testing
diff --git a/packages/agent/src/tools/interaction/agentTools.test.ts b/packages/agent/src/tools/interaction/agentTools.test.ts
@@ -23,7 +23,7 @@ const mockContext: ToolContext = {
   headless: true,
   userSession: false,
   pageFilter: 'none',
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('Agent Tools', () => {
diff --git a/packages/agent/src/tools/interaction/subAgent.test.ts b/packages/agent/src/tools/interaction/subAgent.test.ts
@@ -27,7 +27,7 @@ const mockContext: ToolContext = {
   headless: true,
   userSession: false,
   pageFilter: 'none',
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('subAgentTool', () => {
diff --git a/packages/agent/src/tools/interaction/userPrompt.test.ts b/packages/agent/src/tools/interaction/userPrompt.test.ts
@@ -19,7 +19,7 @@ const mockContext: ToolContext = {
   headless: true,
   userSession: false,
   pageFilter: 'none',
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('userPromptTool', () => {
diff --git a/packages/agent/src/tools/io/textEditor.test.ts b/packages/agent/src/tools/io/textEditor.test.ts
@@ -19,7 +19,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('textEditor', () => {
diff --git a/packages/agent/src/tools/system/respawn.test.ts b/packages/agent/src/tools/system/respawn.test.ts
@@ -13,7 +13,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 describe('respawnTool', () => {
   it('should have correct name and description', () => {
diff --git a/packages/agent/src/tools/system/shellExecute.test.ts b/packages/agent/src/tools/system/shellExecute.test.ts
@@ -13,7 +13,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('shellExecute', () => {
diff --git a/packages/agent/src/tools/system/shellMessage.test.ts b/packages/agent/src/tools/system/shellMessage.test.ts
@@ -15,7 +15,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 
 // Helper function to get instanceId from shellStart result
diff --git a/packages/agent/src/tools/system/shellStart.test.ts b/packages/agent/src/tools/system/shellStart.test.ts
@@ -14,7 +14,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 describe('shellStartTool', () => {
   beforeEach(() => {
diff --git a/packages/agent/src/tools/system/sleep.test.ts b/packages/agent/src/tools/system/sleep.test.ts
@@ -13,7 +13,7 @@ const toolContext: ToolContext = {
   userSession: false,
   pageFilter: 'simple',
   tokenTracker: new TokenTracker(),
-  githubMode: false,
+  githubMode: true,
 };
 
 describe('sleep tool', () => {
diff --git a/packages/cli/src/settings/config.ts b/packages/cli/src/settings/config.ts
@@ -43,7 +43,7 @@ const projectConfigFile = (): string => getProjectConfigFile();
 // Default configuration
 const defaultConfig = {
   // Add default configuration values here
-  githubMode: false,
+  githubMode: true,
   headless: true,
   userSession: false,
   pageFilter: 'none' as 'simple' | 'none' | 'readability',
diff --git a/packages/cli/tests/commands/config.test.ts b/packages/cli/tests/commands/config.test.ts
@@ -69,18 +69,18 @@ describe('Config Command', () => {
       warn: vi.fn(),
     };
     vi.mocked(Logger).mockImplementation(() => mockLogger as unknown as Logger);
-    vi.mocked(getConfig).mockReturnValue({ githubMode: false });
+    vi.mocked(getConfig).mockReturnValue({ githubMode: true });
     vi.mocked(getDefaultConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       customPrompt: '',
     });
     vi.mocked(updateConfig).mockImplementation((config) => ({
-      githubMode: false,
+      githubMode: true,
       ...config,
     }));
     vi.mocked(getConfigAtLevel).mockReturnValue({});
-    vi.mocked(clearConfigKey).mockImplementation(() => ({ githubMode: false }));
-    vi.mocked(clearConfigKey).mockImplementation(() => ({ githubMode: false }));
+    vi.mocked(clearConfigKey).mockImplementation(() => ({ githubMode: true }));
+    vi.mocked(clearConfigKey).mockImplementation(() => ({ githubMode: true }));
   });
 
   afterEach(() => {
@@ -107,13 +107,13 @@ describe('Config Command', () => {
   it('should filter out invalid config keys in list command', async () => {
     // Mock getConfig to return config with invalid keys
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       invalidKey: 'some value',
     } as any);
 
     // Mock getDefaultConfig to return only valid keys
     vi.mocked(getDefaultConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
     });
 
     await command.handler!({
@@ -249,13 +249,13 @@ describe('Config Command', () => {
   it('should clear a configuration value', async () => {
     // Mock getConfig to include the key we want to clear
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       customPrompt: 'custom value',
     });
 
     // Mock getDefaultConfig to include the key we want to clear
     vi.mocked(getDefaultConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       customPrompt: '',
     });
 
@@ -333,7 +333,7 @@ describe('Config Command', () => {
 
   it('should handle non-existent key for clear command', async () => {
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
     });
 
     await command.handler!({
@@ -370,13 +370,13 @@ describe('Config Command', () => {
   it('should list all configuration values with default indicators', async () => {
     // Mock getConfig to return a mix of default and custom values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false, // default value
+      githubMode: true, // default value
       customPrompt: 'custom value', // custom value
     });
 
     // Mock getDefaultConfig to return the default values
     vi.mocked(getDefaultConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       customPrompt: '',
     });
 
diff --git a/packages/cli/tests/settings/config-defaults.test.ts b/packages/cli/tests/settings/config-defaults.test.ts
@@ -54,7 +54,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should use config values for headless, userSession, and pageFilter when not provided in args', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true,
       userSession: false,
       pageFilter: 'none',
@@ -96,7 +96,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should use command line args for headless, userSession, and pageFilter when provided', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true, // Default is true
       userSession: false, // Default is false
       pageFilter: 'none', // Default is none
@@ -132,7 +132,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should test the actual toolAgent call with config defaults', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true,
       userSession: false,
       pageFilter: 'none',
@@ -180,7 +180,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should test the actual toolAgent call with command line args', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true, // Default is true
       userSession: false, // Default is false
       pageFilter: 'none', // Default is none
diff --git a/packages/cli/tests/settings/config.test.ts b/packages/cli/tests/settings/config.test.ts
@@ -61,7 +61,7 @@ describe('Config', () => {
 
   describe('updateConfig', () => {
     it('should update config and write to file', () => {
-      const currentConfig = { githubMode: false };
+      const currentConfig = { githubMode: true };
       const newConfig = { githubMode: true };
       vi.mocked(fs.existsSync).mockReturnValue(true);
       vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(currentConfig));
@@ -77,7 +77,7 @@ describe('Config', () => {
     });
 
     it('should merge partial config with existing config', () => {
-      const currentConfig = { githubMode: false, existingSetting: 'value' };
+      const currentConfig = { githubMode: true, existingSetting: 'value' };
       const partialConfig = { githubMode: true };
       vi.mocked(fs.existsSync).mockReturnValue(true);
       vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(currentConfig));
diff --git a/packages/cli/tests/settings/configDefaults.test.ts b/packages/cli/tests/settings/configDefaults.test.ts
@@ -54,7 +54,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should use config values for headless, userSession, and pageFilter when not provided in args', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true,
       userSession: false,
       pageFilter: 'none',
@@ -90,7 +90,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should use command line args for headless, userSession, and pageFilter when provided', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true, // Default is true
       userSession: false, // Default is false
       pageFilter: 'none', // Default is none
@@ -126,7 +126,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should test the actual toolAgent call with config defaults', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true,
       userSession: false,
       pageFilter: 'none',
@@ -174,7 +174,7 @@ describe('Config Defaults for CLI Options', () => {
   it('should test the actual toolAgent call with command line args', async () => {
     // Setup mock config with default values
     vi.mocked(getConfig).mockReturnValue({
-      githubMode: false,
+      githubMode: true,
       headless: true, // Default is true
       userSession: false, // Default is false
       pageFilter: 'none', // Default is none
