Uh oh!
There was an error while loading. Please reload this page.
SP-1173: extend CUI marking to directory exports - #413
Open
Dennis Woditsch (dwoditsch) wants to merge 4 commits into
Open
Conversation
This was referenced Aug 11, 2026
Dennis Woditsch (dwoditsch)
marked this pull request as ready for review
August 11, 2026 09:45
A directory has no single file to rename, so a classified export keeps its tree and gains the cover sheet inside a prefixed directory. The cover decision helper now takes a writer callback, which lets the three directory commands reuse the same branch the file writers use. extractExportedZipWithNestedZipsToDir resolves its target against the working directory like its sibling extract helpers, so the cover sheet lands next to the tree it belongs to. Includes-AI-Code: true Co-authored-by: Cursor <cursoragent@cursor.com>
Unit cases for the three cover outcomes plus the fail-closed path, and one command case per call site asserting the prefixed directory holds the original tree next to the cover sheet. Includes-AI-Code: true Co-authored-by: Cursor <cursoragent@cursor.com>
pull and pullFile resolve with nothing, so Promise<any> claimed more than they deliver. Every caller awaits and discards the result, and no subclass overrides either method. Includes-AI-Code: true Co-authored-by: Cursor <cursoragent@cursor.com>
Dennis Woditsch (dwoditsch)force-pushed
the
feat/SP-1173-cui-marking-directory-exports
branch
from
August 11, 2026 11:30
e53a81a to
c102e00CompareInternal overview of cover-response classification and per-trigger examples for unclassified and classified artifacts. Includes-AI-Code: true Co-authored-by: Cursor <cursoragent@cursor.com>
Nikolai Fischer (nikolai-celonis)
approved these changes
Aug 11, 2026
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 freeto 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.
Description
Stacked on #412, which finished the single-file row. This PR takes the last one: the three exports whose artifact is a directory tree.
A directory has no single file to rename into
CUI - <name>.zip, so it follows the policy the stack has carried since #405: prefix the directory name and put the cover sheet inside it. A classified export becomesCUI - <name>/holding the original tree next toCUI_Cover_Sheet.pdf, and an unclassified one is renamed only, the same as files.This closes the table. Every user-facing write in the CLI now goes through
CuiFileService.Scope: how the write is triggered
--jsonlistings and reportslist spaces,list packages--jsonlistings and reportslist assets/assignments/data-pools,config *,t2tc package list/diff,deployment *,asset-registry *-o, --outputToJsonFilereportsanalyze/import action-flows,export data-pool,import data-pools,t2tc package importreportconfig package export --zip,config branch export --zip,t2tc package export,export action-flows,pull packagepull asset/skill/data-pool/view-bookmarks/bookmarks,export bookmarksconfig package export,config branch export,t2tc package export --unzip--gitBranchvariantsconfig package export,config branch export,t2tc package exportRelevant links
Checklist
Made with Cursor