Non SFDX File in Package Directory Breaks Clean about force-source-clean

[ralphcallaway]

it also deletes the old src/package.xml manifest, which should go anyway ...

perhaps we make that the description, cleans folder so that it only contains sfdx metadata that are in the current default org

[ralphcallaway]

added a .forceignore with the following and the method runs, but the file gets deleted (fine in this case)

a lot of this boils down to is there a good reason to have non-sfdx files in package folders (kind of feel like now, let's be opinionated and say if it doesn't exist in the org it's deleted, regardless of whether it's an sfdx file our not. package folders are just for sfdx files

[ChuckJonas]

Ya, looks like the error.

I think we could exclude . files and .forceIgnore. EVERYTHING else will be removed. As you saw before, nothing else should be in the source folders as it will break the standards sfdx force:source commands... So by deleting everything else it really is "cleaning up"

[ChuckJonas]

I think the only TODO from this issue is just to make this behavior more apparent in the command help and readme.md

[ralphcallaway]

mostly, i'd say #6 is a todo that comes out of this, but yeah it seems like you'd expect clean to delete these files and to mirror the current constraints of the sfdx command