-
-
Notifications
You must be signed in to change notification settings - Fork 22.9k
Chore: Enhanced Controlled Export/Import Process with Intelligent Conflict Management #5352
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
+2,182
−52
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
a2f96de
Add import conflict preview and update handling
iokinpardo 22b3eb1
Merge pull request #1 from iokinpardo/codex/modify-import-system-for-...
iokinpardo a329306
Fix assistant conflict type mapping
iokinpardo c19e506
Merge branch 'main' into codex/modify-import-system-for-conflict-reso...
iokinpardo 2ff842d
Merge pull request #2 from iokinpardo/codex/modify-import-system-for-...
iokinpardo 21e9b3d
Fix duplicate assistant type mapping declaration
iokinpardo d181085
Merge pull request #3 from iokinpardo/codex/solucionar-problema-de-de...
iokinpardo 5da3699
Enhance import conflict review UI
iokinpardo f8e4fb1
Merge pull request #4 from iokinpardo/codex/apply-adjustments-to-flow...
iokinpardo 5d1462c
Add credential binding metadata to workspace exports
iokinpardo 366f113
Merge pull request #5 from iokinpardo/codex/confirm-credential-export...
iokinpardo 1c71941
feat: improve import review with tabs and summary
iokinpardo 0ebcfae
Merge pull request #6 from iokinpardo/codex/add-tabs-for-import-overv...
iokinpardo 3a6965c
Fix duplicate document store sections when switching tabs
iokinpardo b476df1
Merge pull request #7 from iokinpardo/codex/fix-incorrect-document-st...
iokinpardo 6cf7d9e
feat(ui): add import review accordions
iokinpardo a097e48
Merge pull request #8 from iokinpardo/codex/set-default-checkbox-stat...
iokinpardo 899fd79
Improve import dialog visibility and summary
iokinpardo 437d47e
Merge pull request #9 from iokinpardo/codex/improve-visibility-of-dis...
iokinpardo 49ac7c6
Fix conflict selection keys and expand new item accordions
iokinpardo 09ee16f
Merge pull request #10 from iokinpardo/codex/fix-selection-issue-in-c...
iokinpardo 57f725a
Fix import review new items including conflicts
iokinpardo 52f1a36
Merge pull request #11 from iokinpardo/codex/verify-item-duplication-...
iokinpardo 1e4e080
Fix chatflow credential bindings during import
iokinpardo 6252a61
Merge pull request #12 from iokinpardo/codex/fix-chatflows-credential...
iokinpardo 94bf351
Adjust duplicate import handling and new item UI
iokinpardo ad9cca9
Merge pull request #13 from iokinpardo/codex/add-clone-suffix-to-dupl...
iokinpardo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
21 changes: 21 additions & 0 deletions
docs/credential-export-options.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| # Exportación e importación de credenciales en Agentflow V2 | ||
|
|
||
| Esta guía describe el flujo actual para preservar los nombres de las credenciales al exportar flujos de Agentflow V2 (y otros `ChatFlow`) y cómo se reconstruyen automáticamente al importarlos. | ||
|
|
||
| ## Qué se incluye en la exportación | ||
|
|
||
| * `generateExportFlowData` continúa eliminando cualquier clave `FLOWISE_CREDENTIAL_ID` de los nodos para no exponer IDs sensibles en el archivo descargado.【F:packages/ui/src/utils/genericHelper.js†L551-L607】 | ||
| * Antes de que los datos lleguen al cliente, `exportImportService.exportData` inspecciona cada `ChatFlow`, identifica los nodos que referencian credenciales y adjunta una sección `credentialBindings` con pares `{ nodeId, path, credentialName, credentialType }`. Sólo se incluyen entradas cuyo nombre y tipo pudieron resolverse en la base de datos.【F:packages/server/src/services/export-import/index.ts†L224-L327】【F:packages/server/src/services/export-import/index.ts†L180-L222】 | ||
| * Al exportar un workspace, la sección `credentialBindings` se conserva dentro del JSON de cada flujo, junto con la lista de nodos y edges ya sanitizados.【F:packages/ui/src/utils/exportImport.js†L22-L30】【F:packages/ui/src/utils/genericHelper.js†L559-L606】 | ||
|
|
||
| ## Cómo se reconstruyen al importar | ||
|
|
||
| * Durante la importación, `exportImportService.importData` busca la sección `credentialBindings` en cada flujo y consulta las credenciales disponibles en el workspace de destino (incluyendo compartidas) por nombre y tipo.【F:packages/server/src/services/export-import/index.ts†L329-L433】 | ||
| * Si encuentra una coincidencia, inserta el nuevo ID bajo la clave `FLOWISE_CREDENTIAL_ID` en el nodo indicado y elimina la sección `credentialBindings` antes de persistir el flujo.【F:packages/server/src/services/export-import/index.ts†L374-L433】 | ||
| * Cuando no existe una credencial con el mismo nombre y tipo, se registra una advertencia en los logs para que el usuario pueda completar la vinculación de forma manual tras la importación.【F:packages/server/src/services/export-import/index.ts†L356-L433】 | ||
|
|
||
| ## Notas y limitaciones | ||
|
|
||
| * Sólo se generan entradas en `credentialBindings` para credenciales resolubles; si un flujo hacía referencia a una credencial eliminada, no habrá mapeo automático y el nodo seguirá requiriendo configuración manual al importarse. | ||
| * El mecanismo confía en que los nombres de credencial y sus tipos (`credentialName`) coincidan exactamente entre origen y destino. Diferencias en mayúsculas/minúsculas o en el tipo impedirán la reconexión automática. | ||
| * Las exportaciones individuales desde el canvas no incluyen `credentialBindings`; el mapeo automático sólo se aplica a las exportaciones de workspace. |
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
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
Oops, something went wrong.
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.