Unable to update `.idea/**` Intellij config files, as those are excluded by default without override

[philippe-granet]

What problem are you trying to solve?

We want to update with Openrewrite our Code Style and Copyright on all our Intellij projects, but Openrewrite exclude .idea/ folder by default :

rewrite-polyglot/src/main/java/org/openrewrite/polyglot/OmniParser.java

Line 83 in ac0f3ad

".idea",

So we can't update Intellij config like these:
https://github.com/JetBrains/intellij-community/blob/master/.idea/codeStyles/Project.xml
https://github.com/JetBrains/intellij-community/blob/master/.idea/copyright/apache_2_license.xml

Describe the solution you'd like

.idea/ folder not excluded by default

Have you considered any alternatives or workarounds?

No workaround found

[timtebeek]

hi! As far as I recall that's the first time it's come up. I'm a bit cautious removing that folder from the default set, as it might then surprise others who had relied on it being there. Have you tried adding a negative exclusion path !.idea/? Sometimes we use such patterns to turn include/exclude around. I hadn't tried that here yet, but figured could be worth trying.

[philippe-granet]

Tests of exclusion are harcoded and there is no config to override DEFAULT_IGNORED_DIRECTORIES:

rewrite-polyglot/src/main/java/org/openrewrite/polyglot/OmniParser.java

Lines 160 to 164 in ac0f3ad

	if (mode.equals(FileMode.TREE) &&
	!isExcluded(path, rootDir) &&
	!DEFAULT_IGNORED_DIRECTORIES.contains(path.getFileName().toString()) &&
	!workingTreeIterator.isEntryIgnored()) {
	walk.enterSubtree();

and here:

rewrite-polyglot/src/main/java/org/openrewrite/polyglot/OmniParser.java

Lines 287 to 294 in ac0f3ad

	private boolean isIgnoredDirectory(Path path, Path rootDir) {
	for (Path pathSegment : rootDir.relativize(path)) {
	if (DEFAULT_IGNORED_DIRECTORIES.contains(pathSegment.toString())) {
	return true;
	}
	}
	return false;
	}

[timtebeek]

Hmm; I don't really see an easy way forward without potentially affecting others; Would it be enough for you to overwrite those files completely, ignoring previous content? If so then https://docs.openrewrite.org/recipes/text/createtextfile is potentially enough.

[philippe-granet]

I try this, it works only if the file don't exist, but if file already exist, file is excluded

[timtebeek]

Ah sorry to hear; then we're all out of simple solutions for this use case and need a structural fix. Open to suggestions, but given that you're the first it's not at the top of the list here, and we'd need a solution that won't negatively affect others.