Requiring multiple non-interface namespaces from another component throws java.lang.ClassCastException

[eval-on-point]

Describe the bug
Requiring multiple non-interface namespaces from another component throws java.lang.ClassCastException.

To Reproduce
This example repository shows a minimal example workspace that reproduces the error with instructions.

Expected behavior
Instead of a stacktrace, I expect poly to give me multiple illegal dependency errors or otherwise not throw an error.

Additional context
Found while trying to reproduce the error described in this Slack thread. This error does not seem to appear on version 0.2.19.

[hiredman]

polylith/components/deps/src/polylith/clj/core/deps/brick_deps/core.clj

Line 58 in adfcaa2

illegal-deps (filter #(illegal-dep? % interface-ns source) deps)]

looks like missing map of :to-brick-id or something similar?

[eval-on-point]

Seems like it may not be reproducible on some systems (see this message in Slack). I've added a nix flake to the repo. This flake creates a devenv shell that reproduces the issue for me.

[tengstrand]

Will have a look.

[tengstrand]

I can't reproduce the error:

(comment
  (def source :src)
  (def interface-ns "top.")
  (def deps [{:from-type "component", :from-ns "interface", :to-type "library", :to-brick-id "component-two", :to-namespace "core"} 
             {:from-type "component", :from-ns "core", :to-type "library", :to-brick-id "component-two", :to-namespace "core"}])
  (filter #(illegal-dep? % interface-ns source) deps)
  #__)

The other two should do map :to-brick-id but not illegal-deps.
I can't see what's wrong here, but maybe you can guide me.

[tengstrand]

Maybe you could clone the polylith repo and then point to it in an alias, which is described here.
If the problem remains, then you have access to the polylith source code, so you can add print statements and the like, as a way to find what the problem is.

[hiredman]

I haven't repo'ed the error, just looking at the code

polylith/components/deps/src/polylith/clj/core/deps/brick_deps/core.clj

Lines 51 to 61 in adfcaa2

	(defn deps-for-source [suffixed-top-ns interface-names base-names brick interface-ns source]
	(let [deps (brick-deps-for-source suffixed-top-ns brick interface-names base-names source)
	interface-deps (map :to-brick-id
	(filter #(interface-dep? % interface-ns source)
	deps))
	base-deps (set/intersection (map :to-brick-id
	(filter base? deps)))
	illegal-deps (filter #(illegal-dep? % interface-ns source) deps)]
	[(unique-ids interface-deps)
	(unique-ids base-deps)
	(unique-ids illegal-deps)]))

deps on line 52 is a sequence of maps, interface-deps (line 53) is a sequence of whatever :to-brick-id is (presumably not a map), same for base-deps(line 56), but illegal-deps (line 58) remains a sequence of maps. so interface-deps sorts fine, base-deps sorts fine, but illegal-deps throws an error when trying to sort it (when it is non-empty).

[tengstrand]

Okay, thanks, now I get it! Will fix!

[eval-on-point]

Confirming that 2fca38b fixes the issue in both of my repos.

[tengstrand]

Thanks for setting up that extra repo!