Skip to content

Add workflow-specific local config directory#13

Merged
markphelps merged 2 commits into
mainfrom
local-config-dir-input
Jun 23, 2026
Merged

Add workflow-specific local config directory#13
markphelps merged 2 commits into
mainfrom
local-config-dir-input

Conversation

@markphelps

Copy link
Copy Markdown
Contributor

Summary

  • Add local-config-dir input to the PR review and issue-health composite actions.
  • Resolve local overrides from the selected target-repo directory instead of always using <repo>/.agents.
  • Document workflow-specific override directories for repos that run multiple agents.

Test Plan

  • node --test test/*.mjs
  • npx -p typescript tsc --noEmit -p tsconfig.json
  • pnpm build

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: comment

This PR introduces workflow-specific local config directories, updating code, documentation, and tests. The implementation fits project conventions and avoids regressions. Only minor suggestions for path concatenation remain; no critical changes required.

workflows/pr-review.ts

  • minor (L83): The new resolveLocalConfigDir function concatenates paths by string manipulation. Prefer using Node's path.join() to handle separators reliably across platforms and avoid subtle bugs.

workflows/issue-health.ts

  • minor (L91): In resolveLocalConfigDir, string concatenation is used for paths. Switch to path.join() as the default for combining filesystem paths to ensure proper handling, especially on Windows or with unusual directory layouts.

🤖 Automated review by the Flipt PR review agent.

@markphelps markphelps merged commit 35b027b into main Jun 23, 2026
1 check passed
@markphelps markphelps deleted the local-config-dir-input branch June 23, 2026 00:13
markphelps added a commit that referenced this pull request Jun 23, 2026
* origin/main:
  Use target repo labels directly (#14)
  Add workflow-specific local config directory (#13)
  chore: update gitignore
  Add Issue Health Check agent (#12)
  fix(pr-review): relax hard timeout budget
  Budget PR review effort by PR size (#11)
  feat(pr-review): update existing bot review comment (#10)
  chore: update gitignore
  perf(pr-review): make reviews more decisive to cut runtime and tokens
  refactor: read per-repo overrides from .agents/ instead of .flue/
  feat(models): document Kimi K2.6 on Cloudflare Workers AI (#9)
  feat: brand the review footer as Flipt and link the agents repo
  feat: factor prior PR reviews/comments into the review (avoid repeats, respect decisions) (#8)
  feat: connect the Flipt docs MCP so the reviewer can query the docs (#7)
  docs: reposition README as the Flipt agents fleet (PR Review = first agent) (#6)
  fix: post reviews deterministically from the workflow (not the model) (#5)
  fix: drop setup-node pnpm cache so the action works when consumed cross-repo (#4)
  fix: make the PR review agent run end-to-end (gh token + gpt-5-mini) (#3)
  feat: run on GitHub Models with a free default; clarify AGENTS.md (#1)
  feat: initial commit

# Conflicts:
#	lib/issue-health.ts
#	skills/issue-health/SKILL.md
#	test/issue-health.test.mjs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant