00:00:00

Reject dependency cycles and roll back partial build artifacts

Review the independently proposed patch below, including its unchanged callers. A dependency cycle must fail during planning. Separately, failure after building an intermediate target must leave no intermediate artifact or cache record behind.

Review focus: Content-addressed build caching

src/domain/planner.py

BeforeProposed

+1−2

1010 def visit(name):

1111 if name in visited:

1212 return

13 if name in active:

14 return

13 require(name not in active, 'dependency-cycle')

1514 active.add(name)

1615 definition = target(state, tenant, name)

1716 for item in definition['inputs']:

Your review

0 / 10,000 characters · A verdict needs supporting evidence.

30 more characters to give a verdict