Skip to content

Add workflow orchestrator enhancements - #5

Open
cryptosebek wants to merge 1 commit into
hlibr:masterfrom
cryptosebek:feature/workflow-orchestrator-enhancements
Open

Add workflow orchestrator enhancements#5
cryptosebek wants to merge 1 commit into
hlibr:masterfrom
cryptosebek:feature/workflow-orchestrator-enhancements

Conversation

@cryptosebek

@cryptosebek cryptosebek commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR makes the workflow orchestrator usable directly from an installed Pi package while keeping project-level customization available.

Runtime changes

  • Resolve workflows and bundled tool extensions from the package when a project override is not present.
  • Materialize missing workflow and agent defaults into the project on first use without overwriting existing files.
  • Discover bundled agents when no project agent directory exists, while preserving user/project precedence.
  • Add shorthand command support: /workflow "goal", optional workflow names, quoted arguments, and --model <id>.
  • Persist the selected run model and fall back to the active Pi model when resuming older state. Agent frontmatter models remain higher priority.
  • Resolve configured extension paths against the project or package installation.
  • Improve subagent model-error messages with actionable /model and agent-frontmatter hints.
  • Surface startup and resume configuration errors as workflow notifications instead of unhandled rejections.

Documentation and configuration

  • Update bundled PM, developer, and verifier model defaults to openrouter/free.
  • Update the project guide and README for package defaults, shorthand commands, and model overrides.
  • Refresh development dependency versions.

Tests

  • Add coverage for package fallback, default materialization, extension resolution, command tokenization, model selection, and model-error formatting.
  • Retain and extend workflow clarification regression coverage.
  • Remove duplicate extension-resolution and model-selection tests.

Validation

  • 207 tests passing
  • TypeScript typecheck passing
  • ESLint passing
  • Prettier checks passing
  • git diff --check passing

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d916fdd5b8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

sessionFile,
systemPrompt: agent.systemPrompt,
model: agent.model,
model: pickModel(agent.model, currentState?.model, ctx.model?.id),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Honor the workflow model override before agent defaults

When a user starts /workflow --model <id> "goal", startWorkflow stores that model in currentState.model, but this call passes agent.model to pickModel first. Because the bundled PM/developer/verifier agent files all define model: openrouter/free, the requested run model is never used for those subagents (the PM runner uses the same precedence below), so the new CLI flag silently has no effect unless users edit or remove every agent model line.

Useful? React with 👍 / 👎.

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