gaunt-sloth-assistant 0.4.2 → 0.5.1

package/src/modules/reviewModule.ts

@@ -1,22 +1,25 @@
-import { slothContext } from '#src/config.js';
+import type { SlothConfig } from '#src/config.js';
 import { display, displayDebug, displayError, displaySuccess } from '#src/consoleUtils.js';
-import { stdout } from '#src/systemUtils.js';
 import { generateStandardFileName, ProgressIndicator } from '#src/utils.js';
 import { writeFileSync } from 'node:fs';
 import { invoke } from '#src/llmUtils.js';
 import { getGslothFilePath } from '#src/filePathUtils.js';
 
-export async function review(source: string, preamble: string, diff: string): Promise<void> {
-  const progressIndicator = new ProgressIndicator('Reviewing.');
-  const outputContent = await invoke(slothContext.config.llm, slothContext.session, preamble, diff);
-  progressIndicator.stop();
+export async function review(
+  source: string,
+  preamble: string,
+  diff: string,
+  config: SlothConfig,
+  command: 'pr' | 'review' = 'review'
+): Promise<void> {
+  const progressIndicator = config.streamOutput ? undefined : new ProgressIndicator('Reviewing.');
+  const outputContent = await invoke(config.llm, preamble, diff, config, command);
+  progressIndicator?.stop();
   const filename = generateStandardFileName(source);
   const filePath = getGslothFilePath(filename);
-  stdout.write('\n');
-  display(`writing ${filePath}`);
-  stdout.write('\n');
-  // TODO highlight LLM output with something like Prism.JS (maybe system emoj are enough ✅⚠️❌)
-  display(outputContent);
+  if (!config.streamOutput) {
+    display('\n' + outputContent);
+  }
   try {
     writeFileSync(filePath, outputContent);
     displaySuccess(`This report can be found in ${filePath}`);

package/src/prompt.ts

@@ -5,7 +5,7 @@ import {
 } from '#src/utils.js';
 import { displayError } from '#src/consoleUtils.js';
 import { exit } from '#src/systemUtils.js';
-import { GSLOTH_BACKSTORY } from '#src/config.js';
+import { GSLOTH_BACKSTORY, GSLOTH_SYSTEM_PROMPT } from '#src/constants.js';
 import { getGslothConfigReadPath } from '#src/filePathUtils.js';
 
 export function readBackstory(): string {
@@ -28,6 +28,10 @@ export function readReviewInstructions(reviewInstructions: string): string {
   return readConfigPromptFile(reviewInstructions);
 }
 
+export function readSystemPrompt(): string {
+  return readFileFromCurrentOrInstallDir(GSLOTH_SYSTEM_PROMPT, true);
+}
+
 function readConfigPromptFile(guidelinesFilename: string): string {
   try {
     const filePath = getGslothConfigReadPath(guidelinesFilename);

package/src/providers/ghIssueProvider.ts

@@ -1,5 +1,5 @@
 import { displayWarning } from '#src/consoleUtils.js';
-import { execAsync } from '#src/utils.js';
+import { execAsync, ProgressIndicator } from '#src/utils.js';
 import type { ProviderConfig } from './types.js';
 
 /**
@@ -19,7 +19,9 @@ export async function get(
 
   try {
     // Use the GitHub CLI to fetch issue details
+    const progress = new ProgressIndicator(`Fetching GitHub issue #${issueId}`);
     const issueContent = await execAsync(`gh issue view ${issueId}`);
+    progress.stop();
 
     if (!issueContent) {
       displayWarning(`No content found for GitHub issue #${issueId}`);

package/src/providers/ghPrDiffProvider.ts

@@ -1,5 +1,5 @@
 import { displayWarning } from '#src/consoleUtils.js';
-import { execAsync } from '#src/utils.js';
+import { execAsync, ProgressIndicator } from '#src/utils.js';
 import type { ProviderConfig } from './types.js';
 
 /**
@@ -19,7 +19,9 @@ export async function get(
 
   try {
     // Use the GitHub CLI to fetch PR diff
+    const progress = new ProgressIndicator(`Fetching GitHub PR #${prId} diff`);
     const prDiffContent = await execAsync(`gh pr diff ${prId}`);
+    progress.stop();
 
     if (!prDiffContent) {
       displayWarning(`No diff content found for GitHub PR #${prId}`);

package/src/providers/jiraIssueProvider.ts

@@ -1,6 +1,7 @@
 import { display, displayError, displayWarning } from '#src/consoleUtils.js';
 import { env } from '#src/systemUtils.js';
 import type { JiraConfig } from './types.js';
+import { ProgressIndicator } from '#src/utils.js';
 
 interface JiraIssueResponse {
   fields: {
@@ -105,7 +106,6 @@ async function getJiraIssue(config: JiraConfig, jiraKey: string): Promise<JiraIs
   if (config.displayUrl) {
     display(`Loading Jira issue ${config.displayUrl}${jiraKey}`);
   }
-  display(`Retrieving jira from api ${apiUrl.replace(/^https?:\/\//, '')}`);
 
   // This filter will be necessary for V3: `&expand=renderedFields` to convert ADF to HTML
   const filters = '?fields=summary,description'; // Limit JSON to summary and description
@@ -122,10 +122,14 @@ async function getJiraIssue(config: JiraConfig, jiraKey: string): Promise<JiraIs
     'Accept-Language': 'en-US,en;q=0.9', // Prevents errors in other languages
   };
 
+  const progressIndicator = new ProgressIndicator(
+    `Retrieving jira from api ${apiUrl.replace(/^https?:\/\//, '')}`
+  );
   const response = await fetch(apiUrl + filters, {
     method: 'GET',
     headers: headers,
   });
+  progressIndicator.stop();
 
   if (!response?.ok) {
     try {