Compare commits
7 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 1c128b293f | |||
| c08008668b | |||
| 7ac2938570 | |||
| 363e4ecda2 | |||
| 13cc56944f | |||
| 2d91473f6e | |||
| 3937c3bdba |
@@ -9,9 +9,6 @@ GitHub Action for running Claude Code and other agents via Pullfrog.
|
|||||||
```bash
|
```bash
|
||||||
# Install dependencies
|
# Install dependencies
|
||||||
pnpm install
|
pnpm install
|
||||||
|
|
||||||
# Test with default prompt
|
|
||||||
npm run play # Run locally on your machine
|
|
||||||
```
|
```
|
||||||
|
|
||||||
## Testing with play.ts
|
## Testing with play.ts
|
||||||
|
|||||||
+1
-1
@@ -67,7 +67,7 @@ export const claude: Agent = {
|
|||||||
|
|
||||||
// Write tarball to file
|
// Write tarball to file
|
||||||
const fileStream = createWriteStream(tarballPath);
|
const fileStream = createWriteStream(tarballPath);
|
||||||
await pipeline(response.body as any, fileStream);
|
await pipeline(response.body!, fileStream);
|
||||||
log.info(`Downloaded tarball to ${tarballPath}`);
|
log.info(`Downloaded tarball to ${tarballPath}`);
|
||||||
|
|
||||||
// Extract tarball
|
// Extract tarball
|
||||||
|
|||||||
+1
-1
@@ -1 +1 @@
|
|||||||
run npx cowsay "don't eat me"
|
Review https://github.com/pullfrogai/scratch/pull/17
|
||||||
+7794
-6815
File diff suppressed because one or more lines are too long
+3
-3
@@ -16,9 +16,9 @@ export function createMcpConfigs(githubInstallationToken: string): McpConfigs {
|
|||||||
const repoContext = parseRepoContext();
|
const repoContext = parseRepoContext();
|
||||||
const githubRepository = `${repoContext.owner}/${repoContext.name}`;
|
const githubRepository = `${repoContext.owner}/${repoContext.name}`;
|
||||||
|
|
||||||
const serverPath = process.env.GITHUB_ACTION_PATH
|
// In production (GitHub Actions), mcp-server.js is in same directory as entry.js (where this is bundled)
|
||||||
? `${process.env.GITHUB_ACTION_PATH}/mcp-server.js`
|
// In development, server.ts is in the same directory as this file (config.ts)
|
||||||
: fromHere("server.ts");
|
const serverPath = process.env.GITHUB_ACTIONS ? fromHere("mcp-server.js") : fromHere("server.ts");
|
||||||
|
|
||||||
return {
|
return {
|
||||||
[ghPullfrogMcpName]: {
|
[ghPullfrogMcpName]: {
|
||||||
|
|||||||
+2
-5
@@ -4,9 +4,6 @@ import { contextualize, tool } from "./shared.ts";
|
|||||||
|
|
||||||
export const Review = type({
|
export const Review = type({
|
||||||
pull_number: type.number.describe("The pull request number to review"),
|
pull_number: type.number.describe("The pull request number to review"),
|
||||||
event: type
|
|
||||||
.enumerated("APPROVE", "REQUEST_CHANGES", "COMMENT")
|
|
||||||
.describe("'APPROVE', 'REQUEST_CHANGES', or 'COMMENT' (the review action)"),
|
|
||||||
body: type.string
|
body: type.string
|
||||||
.describe(
|
.describe(
|
||||||
"Brief summary or general feedback that doesn't apply to specific code locations. Keep it concise - most feedback should be in the 'comments' array."
|
"Brief summary or general feedback that doesn't apply to specific code locations. Keep it concise - most feedback should be in the 'comments' array."
|
||||||
@@ -45,7 +42,7 @@ export const ReviewTool = tool({
|
|||||||
"IMPORTANT: Use 'comments' array for ALL specific code issues at the line-level. " +
|
"IMPORTANT: Use 'comments' array for ALL specific code issues at the line-level. " +
|
||||||
"Only use 'body' for a brief summary or feedback that doesn't apply to a specific location.",
|
"Only use 'body' for a brief summary or feedback that doesn't apply to a specific location.",
|
||||||
parameters: Review,
|
parameters: Review,
|
||||||
execute: contextualize(async ({ pull_number, event, body, commit_id, comments = [] }, ctx) => {
|
execute: contextualize(async ({ pull_number, body, commit_id, comments = [] }, ctx) => {
|
||||||
// Get the PR to determine the head commit if commit_id not provided
|
// Get the PR to determine the head commit if commit_id not provided
|
||||||
const pr = await ctx.octokit.rest.pulls.get({
|
const pr = await ctx.octokit.rest.pulls.get({
|
||||||
owner: ctx.owner,
|
owner: ctx.owner,
|
||||||
@@ -58,7 +55,7 @@ export const ReviewTool = tool({
|
|||||||
owner: ctx.owner,
|
owner: ctx.owner,
|
||||||
repo: ctx.name,
|
repo: ctx.name,
|
||||||
pull_number,
|
pull_number,
|
||||||
event,
|
event: "COMMENT",
|
||||||
};
|
};
|
||||||
if (body) params.body = body;
|
if (body) params.body = body;
|
||||||
if (commit_id) {
|
if (commit_id) {
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "@pullfrog/action",
|
"name": "@pullfrog/action",
|
||||||
"version": "0.0.85",
|
"version": "0.0.88",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"files": [
|
"files": [
|
||||||
"index.js",
|
"index.js",
|
||||||
|
|||||||
@@ -1,8 +1,10 @@
|
|||||||
[x] add modes to prompt
|
[x] add modes to prompt
|
||||||
[x] progressively update comment
|
[x] progressively update comment
|
||||||
[] don't allow rejecting prs
|
[x] don't allow rejecting prs
|
||||||
[] fix pnpm caching
|
[x] fix pnpm caching
|
||||||
[] try to avoid claude narrating the initial comment
|
|
||||||
[] fix prompt to avoid narration like "I just read all tools from MCP server"
|
[] fix prompt to avoid narration like "I just read all tools from MCP server"
|
||||||
[] investigate including terminal output from bash commands as collapsed groups
|
[] investigate including terminal output from bash commands as collapsed groups
|
||||||
[] avoid exposing env
|
[] avoid exposing env
|
||||||
|
[] test initialization trade offs for pullfrog.yml
|
||||||
|
[] try to find heavy claude code user
|
||||||
|
[] investigate repo config file?
|
||||||
|
|||||||
Reference in New Issue
Block a user