chore(deps): upgrade @objectstack 7.2.0 → 7.4.1 and modernize flows#364
Merged
Conversation
Upgrade all @objectstack/* packages from ^7.2.0 to ^7.4.1 and bring the
app in line with the 7.4 metadata + Flow capability surface.
Breaking changes addressed:
- Approvals (ADR-0019): `ApprovalProcess.create()` and the top-level
`defineStack({ approvals })` field were removed from @objectstack/spec.
Approvals are now `record_change` flows with `approval` nodes. Removed
src/approvals/ and the config field.
- UI routes: the unified Console shell moved to /_console/ (login at
/_console/login); legacy /_studio/ and /_account/ are gone. Updated the
e2e smoke tests and the config comment.
Flow modernization (re-evaluated all flows against 7.4 capabilities):
- The `script` node with `actionType:'email'` is a silent no-op in 7.4.
Migrated every notification step to the `notify` node (ADR-0012), which
actually dispatches via the messaging service (inbox + email):
lead-conversion, case-escalation, quote-generation.
- opportunity-approval: replaced the dead `connector_action`/`connectorId:
'approval'` pattern (and its `as any` casts) with native `approval` nodes,
and merged the previously separate >$100K and >$500K approval flows into a
single tiered flow — manager review > $100K, director sign-off > $500K via
a `decision` gate — eliminating double-firing on large deals.
Verified: typecheck + build pass (5 flows, protocol-compliant), server boots
clean on 7.4.1, e2e suite green (the two opportunity-lifecycle tests self-skip
under the auth-gated data API, as designed).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Upgrades all
@objectstack/*packages from^7.2.0to^7.4.1and brings HotCRM in line with the 7.4 metadata + Flow capability surface.Breaking changes addressed
ApprovalProcess.create()and the top-leveldefineStack({ approvals })field were removed from@objectstack/spec. Approvals are nowrecord_changeflows withapprovalnodes. Removedsrc/approvals/and the config field./_console/(login at/_console/login); legacy/_studio/and/_account/are gone. Updated the e2e smoke tests and the config comment.Flow modernization (re-evaluated every flow against 7.4 capabilities)
scriptnode withactionType:'email'is a silent no-op in 7.4. Migrated every notification step to thenotifynode (ADR-0012), which dispatches via the messaging service (inbox + email):lead-conversion,case-escalation,quote-generation.opportunity-approval: replaced the deadconnector_action/connectorId:'approval'pattern (and itsas anycasts) with nativeapprovalnodes, and merged the previously separate >$100K and >$500K approval flows into a single tiered flow — manager review > $100K, director sign-off > $500K via adecisiongate — eliminating double-firing on large deals.Verification
pnpm typecheck✓pnpm build✓ — 5 flows, protocol-compliant artifactpnpm test:e2e✓ — 4 passed, 2 skipped (the opportunity-lifecycle tests self-skip under the auth-gated data API, as designed)🤖 Generated with Claude Code