Skip to content

Relocate root configs/assets and add trajectories#1008

Merged
willwashburn merged 2 commits into
mainfrom
simplify
May 27, 2026
Merged

Relocate root configs/assets and add trajectories#1008
willwashburn merged 2 commits into
mainfrom
simplify

Conversation

@willwashburn
Copy link
Copy Markdown
Member

Clean up the repository root by moving and removing several root-level configs and assets: move .gitleaks.toml -> .github/gitleaks.toml and update the security workflow; move readme-banner.png -> web/public/readme-banners/relay.png and update README and package READMEs to reference the hosted image; relocate .eslintrc.cjs into packages/cli; remove .dockerignore, .env.example, .gitattributes, and .mcp.json; and update AGENTS.md to use the agent-trajectories npx command. Also add multiple .agentworkforce trajectory artifacts (summaries, traces, and trajectory JSON) that document audits and the moves performed.

Clean up the repository root by moving and removing several root-level configs and assets: move .gitleaks.toml -> .github/gitleaks.toml and update the security workflow; move readme-banner.png -> web/public/readme-banners/relay.png and update README and package READMEs to reference the hosted image; relocate .eslintrc.cjs into packages/cli; remove .dockerignore, .env.example, .gitattributes, and .mcp.json; and update AGENTS.md to use the agent-trajectories npx command. Also add multiple .agentworkforce trajectory artifacts (summaries, traces, and trajectory JSON) that document audits and the moves performed.
@willwashburn willwashburn requested a review from khaliqgant as a code owner May 27, 2026 15:58
@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This PR records five completed AI-guided workflow trajectories documenting root directory cleanup audit, ESLint config relocation, README banner asset move, Gitleaks config migration, and Agent Relay MCP inspection. It updates the Gitleaks workflow path, removes a beads merge driver rule, and migrates README banner references to hosted assets while updating trajectory command documentation.

Changes

Repository Cleanup and Migration

Layer / File(s) Summary
AI Workflow Trajectory Records
.agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre/..., .agentworkforce/trajectories/completed/2026-05/traj_fvgpb6tvjygd/..., .agentworkforce/trajectories/completed/2026-05/traj_rctcvl6hu15c/..., .agentworkforce/trajectories/completed/2026-05/traj_termzdfoavap/..., .agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk/...
Five completed AI task trajectories are recorded with trace, summary, and JSON metadata files. Tasks document root cleanup audit, ESLint config move to packages/cli, README banner asset relocation, Gitleaks config migration to .github/, and Agent Relay MCP callback support verification.
Configuration Path Updates and Cleanup
.github/workflows/security.yml, .gitattributes
Gitleaks workflow updated to read config from .github/gitleaks.toml instead of repository root. Beads merge driver rule removed from .gitattributes.
Documentation and Reference Updates
AGENTS.md, README.md, packages/agent/README.md, packages/cli/README.md, packages/events/README.md
README banner image URLs migrated from local ./readme-banner.png to hosted https://agentrelay.com/readme-banners/relay.png across root and package documentation. Trajectory setup command updated from npx trail start to npx --yes agent-trajectories start.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

  • khaliqgant

Poem

🐰 Five journeys logged, each cleanup neat,
Configs migrated, paths complete,
Banners now dance on hosted skies,
While .gitattributes says goodbye,
The relay mcp's story to rely!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides good detail about specific changes but lacks the required Summary, Test Plan, and Screenshots sections from the template. Add missing sections: include a brief summary under the Summary heading, complete the Test Plan checklist, and add Screenshots section if applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: relocating root-level configs/assets and adding trajectory documentation artifacts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch simplify

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

Preview deployed!

Environment URL
Web https://d7z2vgjvgaw9i.cloudfront.net

This preview will be cleaned up when the PR is merged or closed.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 5

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
@.agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre/trajectory.json:
- Line 56: The tracked artifact contains a machine-local absolute path in the
JSON key "projectId"; replace that value with a repo-relative identifier or an
opaque project ID (or remove the key entirely) so no developer-local filesystem
paths are committed — update the "projectId" value in trajectory.json (the
"projectId" field) to a safe, non-absolute string or delete the field and commit
the change.

In
@.agentworkforce/trajectories/completed/2026-05/traj_fvgpb6tvjygd/trajectory.json:
- Line 19: The committed trajectory.json contains a developer-local absolute
path in the "projectId" field; replace that value with a non-identifying
placeholder or a relative/portable value (e.g., "projectId": "relay" or remove
the "projectId" field) to avoid exposing local filesystem paths; update the
"projectId" entry in trajectory.json (the "projectId" key) to the sanitized
value and re-run any serialization/generation step that writes
.agentworkforce/trajectories so the change persists.

In
@.agentworkforce/trajectories/completed/2026-05/traj_rctcvl6hu15c/trajectory.json:
- Line 19: The committed trajectory metadata contains a local absolute path in
the JSON key "projectId" inside trajectory.json which leaks a workstation path;
update this by either removing the "projectId" field entirely or replacing its
value with an anonymized repo-scoped identifier (e.g., "repo:relay" or a short
UUID) so no local user/machine information is stored, then amend the commit or
create a new commit that replaces the literal "/Users/..." value with the chosen
anonymized value and verify the file still parses as valid JSON.

In
@.agentworkforce/trajectories/completed/2026-05/traj_termzdfoavap/trajectory.json:
- Line 19: The committed trajectory metadata contains a host-local absolute path
in the "projectId" field which leaks a username and environment-specific data;
update the "projectId" value in trajectory.json (the "projectId" key) to a
repo-relative identifier or an opaque ID (e.g., "relay" or a UUID) instead of
"/Users/will/…", ensuring no user-specific absolute paths remain in committed
metadata.

In
@.agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk/trajectory.json:
- Line 22: The committed JSON contains a local absolute path in the "projectId"
field which leaks user/workstation details; update the "projectId" value in
trajectory.json (the "projectId" key) to a stable repo identifier (e.g., a
repository name or UUID), a relative/portable path, or remove the field entirely
before committing, and ensure any generation step or script that writes
"projectId" uses a CI-safe value (env var or repo metadata) rather than the
local absolute path.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 92b7287d-7fff-41e5-b393-5bda8b70c4c9

📥 Commits

Reviewing files that changed from the base of the PR and between b29b6a1 and aa0e625.

⛔ Files ignored due to path filters (1)
  • web/public/readme-banners/relay.png is excluded by !**/*.png
📒 Files selected for processing (24)
  • .agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre.trace.json
  • .agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre/summary.md
  • .agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre/trajectory.json
  • .agentworkforce/trajectories/completed/2026-05/traj_fvgpb6tvjygd/summary.md
  • .agentworkforce/trajectories/completed/2026-05/traj_fvgpb6tvjygd/trajectory.json
  • .agentworkforce/trajectories/completed/2026-05/traj_rctcvl6hu15c/summary.md
  • .agentworkforce/trajectories/completed/2026-05/traj_rctcvl6hu15c/trajectory.json
  • .agentworkforce/trajectories/completed/2026-05/traj_termzdfoavap/summary.md
  • .agentworkforce/trajectories/completed/2026-05/traj_termzdfoavap/trajectory.json
  • .agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk.trace.json
  • .agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk/summary.md
  • .agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk/trajectory.json
  • .dockerignore
  • .env.example
  • .gitattributes
  • .github/gitleaks.toml
  • .github/workflows/security.yml
  • .mcp.json
  • AGENTS.md
  • README.md
  • packages/agent/README.md
  • packages/cli/.eslintrc.cjs
  • packages/cli/README.md
  • packages/events/README.md
💤 Files with no reviewable changes (4)
  • .dockerignore
  • .gitattributes
  • .mcp.json
  • .env.example

"packages/sdk/src/index.ts",
"packages/sdk/src/relaycast-errors.ts"
],
"projectId": "/Users/will/Projects/AgentWorkforce/relay",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Remove machine-local absolute path from tracked artifact.

Line 56 commits a local filesystem path with a user identifier. Please replace projectId with a repo-relative/opaque identifier (or omit it) to avoid leaking developer-local metadata in version control.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
@.agentworkforce/trajectories/completed/2026-05/traj_a4cddmmx8gre/trajectory.json
at line 56, The tracked artifact contains a machine-local absolute path in the
JSON key "projectId"; replace that value with a repo-relative identifier or an
opaque project ID (or remove the key entirely) so no developer-local filesystem
paths are committed — update the "projectId" value in trajectory.json (the
"projectId" field) to a safe, non-absolute string or delete the field and commit
the change.

},
"commits": [],
"filesChanged": [],
"projectId": "/Users/will/Projects/AgentWorkforce/relay",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Avoid committing local absolute projectId paths.

Line 19 exposes a developer-local path (/Users/will/...). Please sanitize to a non-identifying value (or remove the field) before merge.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
@.agentworkforce/trajectories/completed/2026-05/traj_fvgpb6tvjygd/trajectory.json
at line 19, The committed trajectory.json contains a developer-local absolute
path in the "projectId" field; replace that value with a non-identifying
placeholder or a relative/portable value (e.g., "projectId": "relay" or remove
the "projectId" field) to avoid exposing local filesystem paths; update the
"projectId" entry in trajectory.json (the "projectId" key) to the sanitized
value and re-run any serialization/generation step that writes
.agentworkforce/trajectories so the change persists.

},
"commits": [],
"filesChanged": [],
"projectId": "/Users/will/Projects/AgentWorkforce/relay",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Strip local workstation path from committed trajectory metadata.

Line 19 contains /Users/will/..., which leaks local user/machine info in git history. Replace with an anonymized/repo-scoped identifier or remove projectId.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
@.agentworkforce/trajectories/completed/2026-05/traj_rctcvl6hu15c/trajectory.json
at line 19, The committed trajectory metadata contains a local absolute path in
the JSON key "projectId" inside trajectory.json which leaks a workstation path;
update this by either removing the "projectId" field entirely or replacing its
value with an anonymized repo-scoped identifier (e.g., "repo:relay" or a short
UUID) so no local user/machine information is stored, then amend the commit or
create a new commit that replaces the literal "/Users/..." value with the chosen
anonymized value and verify the file still parses as valid JSON.

},
"commits": [],
"filesChanged": [],
"projectId": "/Users/will/Projects/AgentWorkforce/relay",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Remove host-local absolute path from committed trajectory metadata.

projectId currently includes a local filesystem path with a username (/Users/will/...). This leaks environment-specific identity data into version control and reduces artifact portability. Prefer a repo-relative or opaque identifier.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
@.agentworkforce/trajectories/completed/2026-05/traj_termzdfoavap/trajectory.json
at line 19, The committed trajectory metadata contains a host-local absolute
path in the "projectId" field which leaks a username and environment-specific
data; update the "projectId" value in trajectory.json (the "projectId" key) to a
repo-relative identifier or an opaque ID (e.g., "relay" or a UUID) instead of
"/Users/will/…", ensuring no user-specific absolute paths remain in committed
metadata.

".agentworkforce/trajectories/completed/2026-05/traj_ei1zajpyq584/summary.md",
".agentworkforce/trajectories/completed/2026-05/traj_ei1zajpyq584/trajectory.json"
],
"projectId": "/Users/will/Projects/AgentWorkforce/relay",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Avoid committing local absolute projectId values.

/Users/will/Projects/... exposes local user/workstation details and is not portable across contributors/CI. Use a stable repo identifier (or omit this field in committed artifacts).

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
@.agentworkforce/trajectories/completed/2026-05/traj_xkp9t9riveqk/trajectory.json
at line 22, The committed JSON contains a local absolute path in the "projectId"
field which leaks user/workstation details; update the "projectId" value in
trajectory.json (the "projectId" key) to a stable repo identifier (e.g., a
repository name or UUID), a relative/portable path, or remove the field entirely
before committing, and ensure any generation step or script that writes
"projectId" uses a CI-safe value (env var or repo metadata) rather than the
local absolute path.

@willwashburn willwashburn merged commit 9bd45de into main May 27, 2026
2 checks passed
@willwashburn willwashburn deleted the simplify branch May 27, 2026 16:08
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