Skip to content

Co-locate setup-society composite action with its TS mirror#15

Merged
ericmourant merged 1 commit into
mainfrom
copilot/move-action-yml-to-folder
May 27, 2026
Merged

Co-locate setup-society composite action with its TS mirror#15
ericmourant merged 1 commit into
mainfrom
copilot/move-action-yml-to-folder

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 27, 2026

The setup-society composite action lived under .github/actions/, separated from its canonical TypeScript mirror dispatcher.ts in .github-society-intelligence/lifecycle/society/. Per FINAL.md, those two implementations must move together — keeping them in sibling directories makes that contract obvious and lets the action ship as part of the normal .github-society-intelligence/ install payload.

Changes

  • Action move: .github/actions/setup-society/action.yml.github-society-intelligence/lifecycle/society/setup-society/action.yml.
  • Workflow: updated uses: ./.github/actions/setup-societyuses: ./.github-society-intelligence/lifecycle/society/setup-society in github-society-intelligence-agent.yml.
  • Installer: removed the dedicated cp -R "$EXTRACTED/.github/actions/setup-society" … step and the .github/actions/ entry from the install commit's git add; the action now travels inside the existing .github-society-intelligence/ copy.
  • Docs: updated path references in FINAL.md, FINAL-PLAN.md, PLAN-SINGLE.md, COST-REDUCTION.md.

Note for reviewers

The think.censor.dispatcher-integrity scaffold described in PLAN-SINGLE.md watches setup-society/ for changes paired with governance/dispatcher-fixtures/ updates. If that censor matches on the old path, its glob needs to be retargeted to the new location.

@ericmourant ericmourant marked this pull request as ready for review May 27, 2026 01:25
Copilot AI review requested due to automatic review settings May 27, 2026 01:25
@ericmourant ericmourant merged commit a6ab375 into main May 27, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR relocates the setup-society composite action to live alongside its TypeScript mirror under .github-society-intelligence/, and updates the workflow/docs/installer behavior so the action ships with the standard .github-society-intelligence/ install payload.

Changes:

  • Moved the composite dispatcher action into .github-society-intelligence/lifecycle/society/setup-society/ and updated the workflow to uses: the new path.
  • Simplified the installer by removing the dedicated .github/actions/setup-society copy and .github/actions/ staging.
  • Updated design docs to reference the new canonical dispatcher path.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/github-society-intelligence-agent.yml Updates dispatcher uses: path and adjusts installer/staging behavior.
.github-society-intelligence/lifecycle/society/setup-society/action.yml Adds the relocated composite dispatcher implementation.
.github-society-intelligence/docs/design/PLAN-SINGLE.md Updates references to the dispatcher location.
.github-society-intelligence/docs/design/FINAL.md Updates canonical dispatcher path references.
.github-society-intelligence/docs/design/FINAL-PLAN.md Updates canonical dispatcher path references.
.github-society-intelligence/docs/design/COST-REDUCTION.md Updates path reference for the composite action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 253 to 257
git config user.name "github-actions[bot]"
git config user.email "github-actions[bot]@users.noreply.github.com"

git add .github-society-intelligence/ .github/actions/ .gitignore .gitattributes
git add .github-society-intelligence/ .gitignore .gitattributes

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.

3 participants