Skip to content

fix(greenhouse): align configured agent inputs with API parameters - #9213

Merged
waleedlatif1 merged 11 commits into
stagingfrom
codex/greenhouse-agent-inputs
Oct 19, 2026
Merged

waleedlatif1 merged 11 commits into
stagingfrom
codex/greenhouse-agent-inputs

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Align Greenhouse Harvest v3 controls with tool parameter IDs so agents can configure every supported operation, including conversion targets and date filters.
  • Preserve saved canvas and legacy import values with an operation-scoped migration, remove retired aliases, and clarify inputs shared by list and mutation operations.
  • Evaluate configured-agent dropdown choices using the selected tool operation and parameters; clear incompatible inputs on operation changes while retaining OAuth connections and shared context.
  • Evaluate conditional required fields with the correct canvas, preview, or nested-tool values.

Type of Change

  • Bug fix

Testing

  • Root lint, all 60 audits, all 26 workspace type checks, block-registry validation, and canvas-sentence checks passed.
  • Regenerated tool metadata, integration docs, deployment configuration, and shared documentation artifacts.
  • CI-only regression proof: tests-first run reproduced missing controls and saved-value failures. The option regression run also reproduced all four invalid filter-choice failures. The import regression run reproduced seven saved-value losses. The operation-transition snapshot reproduced all nine intended carryover and credential-loss failures. Full test suites, integration/E2E checks, and static test typing run in CI only.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Oct 19, 2026 1:12am UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

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.

All reported issues were addressed across 7 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/workflows/migrations/greenhouse-agent-inputs.ts
Comment thread apps/sim/blocks/blocks/greenhouse.ts
@greptile-apps

greptile-apps Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no new actionable issues were found.

Summary

Aligns Greenhouse controls with the parameters its tools accept.

  • Greenhouse controls use API parameter names and operation-specific choices.
  • Saved Greenhouse inputs move to the matching API parameters.
  • Agent operation changes keep compatible inputs and OAuth connections.

Previous unnumbered threads were checked: operation-specific choices, incompatible values after operation changes, and preview required markers are addressed. No new actionable issues were found.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Saved Greenhouse block] --> B[Move active legacy inputs to current IDs]
  B --> C[Remove retired aliases]
  C --> D[Serialize current parameters]
  E[Configured agent tool] --> F[Read selected operation and parameters]
  F --> G[Build operation-specific choices]
  F --> H[Show required fields]
  I[User changes operation] --> J[Keep accepted inputs and matching OAuth connection]
  J --> K[Clear operation-dependent inputs]
Loading

Reviews (7) · Last reviewed commit: "fix(editor): preserve preview required f..." · Reviewed by Greptile

Comment thread apps/sim/blocks/blocks/greenhouse.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

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.

All reported issues were addressed across 7 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/workflows/migrations/greenhouse-agent-inputs.ts Outdated
Comment thread apps/sim/blocks/blocks/greenhouse.ts
@waleedlatif1
waleedlatif1 force-pushed the codex/greenhouse-agent-inputs branch from 49cdf2d to 7c3d1ee Compare October 19, 2026 00:22
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 8 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 8 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/greenhouse.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

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.

All reported issues were addressed across 11 files

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/greenhouse.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 12 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 force-pushed the codex/greenhouse-agent-inputs branch from 49165fe to b5b5658 Compare October 19, 2026 01:07
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 12 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 9f8b03d into staging Oct 19, 2026
56 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/greenhouse-agent-inputs branch October 19, 2026 03:21

This branch was successfully deployed

1 active deployment
Preview — b5b5658b Deployed Oct 19, 2026 by vercel[bot]
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