Skip to content

feat: implement fileManager utility with project setup, sanitization#31

Merged
salazarsebas merged 3 commits intoScaffoldRust:mainfrom
pushkarm029:sanitize-file
Aug 8, 2025
Merged

feat: implement fileManager utility with project setup, sanitization#31
salazarsebas merged 3 commits intoScaffoldRust:mainfrom
pushkarm029:sanitize-file

Conversation

@pushkarm029
Copy link
Contributor

Closes #28

  • Implement fileManager utility with project setup, sanitization and cleanup functions
  • Sanitize Directory Names with sanitize-filename
  • Tests

… and cleanup functions

Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com>
@pushkarm029
Copy link
Contributor Author

@salazarsebas please review

…nd index files

Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com>
@pushkarm029
Copy link
Contributor Author

Sorry to bother you—could you approve the workflow? I’ve fixed the formatting. @salazarsebas

…ger tests

- Updated the build script in package.json to specify the target as node.
- Refactored fileManager tests to use 'unknown' type for better type safety when handling null and undefined inputs.

Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com>
@pushkarm029
Copy link
Contributor Author

Fixed

@pushkarm029
Copy link
Contributor Author

CI is green. You can review this now, thanks!

Copy link
Member

@salazarsebas salazarsebas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@salazarsebas salazarsebas merged commit 841ec6b into ScaffoldRust:main Aug 8, 2025
1 check passed
@pushkarm029 pushkarm029 deleted the sanitize-file branch August 8, 2025 08:22
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.

Sanitize Directory Names with sanitize-filename [backend]

2 participants