Repository navigation
Feat/automation clean and transform nodes - #3774
Open
travon-speller wants to merge 12 commits into
Open
travon-speller wants to merge 12 commits into
travon-speller wants to merge 12 commits into
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
3 tasks
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
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.
Description
Adds guided Clean data and Transform data authoring to Automation, with typed Purge filters, operation-specific configuration, upstream table inspection, and improved active-run result selection.
Required backend: SEMOSS/Semoss#3163. Deploy the paired branches and refresh the backend node catalog after restart.
Changes Made
clean.processin the Clean category anddata.transformin Data, using backend catalog definitions.The current merge-base diff is limited to
packages/automation(26 files). It does not add a new service, permission system, frame runtime, or Python-node execution mode.Authoring and Inspection Flow
flowchart TD A[Backend node catalog] --> B[Add Clean or Transform node] B --> C[Choose upstream table and operation] C --> D[Guided fields or typed Purge Builder / JSON] D --> E[Validate and serialize node configuration] E --> F[Existing authorized save and Java execution] C --> G[User clicks Inspect table] G --> H[Read up to 20 recent runs through existing APIs] H --> I{Matching successful root-node table} I -->|Retained result| J[Read schema and bounded result page] I -->|Available live frame| K[Use existing frame pager] I -->|No usable output| L[Show actionable empty state] J --> M[Table viewer; never execute draft nodes] K --> M F --> N[Active run: automatic latest/final selection or explicit user selection]Behavior and Limits
packages/automation/docs/clean-nodes.md; R/Polars and shared BI importer fixes are outside this PR.Validation Status — 2026-10-28
Reviewed head:
74332a94313ea2b500bc42409b9f59acb1e8bd2f.pnpm --filter @semoss/automation test: 18 files / 109 tests passed.@semoss/automationtype-check: not passing (29 diagnostics in unchanged dev audit-log/run-data-browser/viewer files and a local untracked tabular-file form). These diagnostics are not attributed to this PR's changed forms; unrelated files were not modified.Open Review Findings Before Merge
onChangeimmediately removes the row added by Add column, so users cannot enter a second maintained column. A focused review-only test using the actual form/list handlers reproduces the failure. JSON entry is only a workaround.How to Test