[BuildLibrary] whitespace

This commit is contained in:
Jan Petykiewicz 2026-06-19 19:43:21 -07:00
commit 0fba187d8c

View file

@ -598,12 +598,12 @@ class ILibraryView(Mapping[str, 'Pattern'], metaclass=ABCMeta):
for ref_transform in ref_transforms: for ref_transform in ref_transforms:
self.dfs( self.dfs(
pattern=self[target], pattern = self[target],
visit_before=visit_before, visit_before = visit_before,
visit_after=visit_after, visit_after = visit_after,
hierarchy=hierarchy + (target,), hierarchy = hierarchy + (target,),
transform=ref_transform, transform = ref_transform,
memo=memo, memo = memo,
) )
if visit_after is not None: if visit_after is not None:
@ -1622,7 +1622,7 @@ class BuildLibrary(ILibrary):
raise BuildError( raise BuildError(
f'BuildLibrary.{operation}() is only available while validate() or build() is running. ' f'BuildLibrary.{operation}() is only available while validate() or build() is running. '
'Use the built output library for reads.' 'Use the built output library for reads.'
) )
return session return session
def _assert_editable(self) -> None: def _assert_editable(self) -> None:
@ -1742,7 +1742,7 @@ class BuildLibrary(ILibrary):
raise BuildError( raise BuildError(
f'Cannot rename declared build cell "{old_name}" during authoring. ' f'Cannot rename declared build cell "{old_name}" during authoring. '
'Register it under the intended final name instead.' 'Register it under the intended final name instead.'
) )
if old_name not in self._names: if old_name not in self._names:
raise LibraryError(f'"{old_name}" does not exist in the builder.') raise LibraryError(f'"{old_name}" does not exist in the builder.')
if new_name in self._names: if new_name in self._names:
@ -1751,17 +1751,17 @@ class BuildLibrary(ILibrary):
raise BuildError( raise BuildError(
'BuildLibrary.rename(..., move_references=True) is not supported for imported source cells. ' 'BuildLibrary.rename(..., move_references=True) is not supported for imported source cells. '
'Builder-level renames only change the visible imported name.' 'Builder-level renames only change the visible imported name.'
) )
source_index = next( source_index = next(
(idx for idx, spec in enumerate(self._sources) if old_name in spec.visible_to_source), (idx for idx, spec in enumerate(self._sources) if old_name in spec.visible_to_source),
None, None,
) )
if source_index is None: if source_index is None:
raise BuildError( raise BuildError(
f'Cannot rename "{old_name}" during authoring because only imported source-backed ' f'Cannot rename "{old_name}" during authoring because only imported source-backed '
'cells may be renamed on a BuildLibrary.' 'cells may be renamed on a BuildLibrary.'
) )
spec = self._sources[source_index] spec = self._sources[source_index]
source_name = spec.visible_to_source[old_name] source_name = spec.visible_to_source[old_name]
@ -1776,10 +1776,10 @@ class BuildLibrary(ILibrary):
self._sources[source_index] = replace( self._sources[source_index] = replace(
spec, spec,
source_to_visible=source_to_visible, source_to_visible = source_to_visible,
visible_to_source=visible_to_source, visible_to_source = visible_to_source,
order=tuple(order), order = tuple(order),
) )
self._names.remove(old_name) self._names.remove(old_name)
self._names.add(new_name) self._names.add(new_name)
self._order[self._order.index(old_name)] = new_name self._order[self._order.index(old_name)] = new_name
@ -1838,12 +1838,12 @@ class BuildLibrary(ILibrary):
new_names.append(visible) new_names.append(visible)
self._sources.append(_SourceDeclaration( self._sources.append(_SourceDeclaration(
library=view, library = view,
source_to_visible=dict(source_to_visible), source_to_visible = dict(source_to_visible),
visible_to_source=dict(visible_to_source), visible_to_source = dict(visible_to_source),
child_graph={name: set(children) for name, children in child_graph.items()}, child_graph = {name: set(children) for name, children in child_graph.items()},
order=tuple(source_to_visible[name] for name in source_order), order = tuple(source_to_visible[name] for name in source_order),
)) ))
for visible in new_names: for visible in new_names:
self._names.add(visible) self._names.add(visible)
self._order.append(visible) self._order.append(visible)
@ -1943,7 +1943,7 @@ class _BuildSessionLibrary(ILibrary):
self._source_layer_type = _SourceLayer self._source_layer_type = _SourceLayer
self._states: dict[str, Literal['unbuilt', 'building', 'built']] = { self._states: dict[str, Literal['unbuilt', 'building', 'built']] = {
name: 'unbuilt' for name in builder._declarations name: 'unbuilt' for name in builder._declarations
} }
self._declared_stack: list[str] = [] self._declared_stack: list[str] = []
self._emission_stack: list[str] = [] self._emission_stack: list[str] = []
self._emission_via_stack: list[emitted_via_t] = [] self._emission_via_stack: list[emitted_via_t] = []
@ -1957,12 +1957,12 @@ class _BuildSessionLibrary(ILibrary):
def _install_sources(self) -> None: def _install_sources(self) -> None:
for spec in self._builder._sources: for spec in self._builder._sources:
layer = self._source_layer_type( layer = self._source_layer_type(
library=spec.library, library = spec.library,
source_to_visible=dict(spec.source_to_visible), source_to_visible = dict(spec.source_to_visible),
visible_to_source=dict(spec.visible_to_source), visible_to_source = dict(spec.visible_to_source),
child_graph={name: set(children) for name, children in spec.child_graph.items()}, child_graph = {name: set(children) for name, children in spec.child_graph.items()},
order=list(spec.order), order = list(spec.order),
) )
layer_index = len(self._overlay._layers) layer_index = len(self._overlay._layers)
self._overlay._layers.append(layer) self._overlay._layers.append(layer)
source_info = getattr(spec.library, 'library_info', None) source_info = getattr(spec.library, 'library_info', None)
@ -1970,22 +1970,22 @@ class _BuildSessionLibrary(ILibrary):
for source_name, visible_name in spec.source_to_visible.items(): for source_name, visible_name in spec.source_to_visible.items():
self._overlay._entries[visible_name] = self._source_entry_type( self._overlay._entries[visible_name] = self._source_entry_type(
layer_index=layer_index, layer_index = layer_index,
source_name=source_name, source_name = source_name,
) )
if visible_name not in self._overlay._order: if visible_name not in self._overlay._order:
self._overlay._order.append(visible_name) self._overlay._order.append(visible_name)
self._provenance[visible_name] = CellProvenance( self._provenance[visible_name] = CellProvenance(
final_name=visible_name, final_name = visible_name,
requested_name=source_name, requested_name = source_name,
kind='source', kind = 'source',
owner_declared_name=None, owner_declared_name = None,
emitted_via='source_import', emitted_via = 'source_import',
build_chain=(), build_chain = (),
renamed_from=source_name if visible_name != source_name else None, renamed_from = source_name if visible_name != source_name else None,
source_name=source_name, source_name = source_name,
source_metadata=source_meta, source_metadata = source_meta,
) )
def __iter__(self) -> Iterator[str]: def __iter__(self) -> Iterator[str]:
return (name for name in self._order if name in self._names) return (name for name in self._order if name in self._names)
@ -2056,16 +2056,16 @@ class _BuildSessionLibrary(ILibrary):
requested_name = provenance.requested_name requested_name = provenance.requested_name
self._provenance[new_name] = replace( self._provenance[new_name] = replace(
provenance, provenance,
final_name=new_name, final_name = new_name,
renamed_from=requested_name if new_name != requested_name else None, renamed_from = requested_name if new_name != requested_name else None,
) )
owner = provenance.owner_declared_name owner = provenance.owner_declared_name
if owner is not None and owner in self._owned_cells: if owner is not None and owner in self._owned_cells:
self._owned_cells[owner] = [ self._owned_cells[owner] = [
new_name if cell_name == old_name else cell_name new_name if cell_name == old_name else cell_name
for cell_name in self._owned_cells[owner] for cell_name in self._owned_cells[owner]
] ]
return self return self
def __getitem__(self, key: str) -> 'Pattern': def __getitem__(self, key: str) -> 'Pattern':
@ -2102,14 +2102,14 @@ class _BuildSessionLibrary(ILibrary):
self._emission_stack.append(key) self._emission_stack.append(key)
try: try:
self._record_provenance( self._record_provenance(
final_name=key, final_name = key,
requested_name=key, requested_name = key,
kind=kind, kind = kind,
owner_declared_name=current if kind == 'helper' else key, owner_declared_name = current if kind == 'helper' else key,
emitted_via=via, emitted_via = via,
build_chain=tuple(self._declared_stack), build_chain = tuple(self._declared_stack),
renamed_from=None, renamed_from = None,
) )
finally: finally:
self._emission_stack.pop() self._emission_stack.pop()
@ -2150,10 +2150,10 @@ class _BuildSessionLibrary(ILibrary):
if new_name in self._provenance: if new_name in self._provenance:
self._provenance[new_name] = replace( self._provenance[new_name] = replace(
self._provenance[new_name], self._provenance[new_name],
requested_name=old_name, requested_name = old_name,
renamed_from=old_name, renamed_from = old_name,
owner_declared_name=current if current is not None else self._provenance[new_name].owner_declared_name, owner_declared_name = current if current is not None else self._provenance[new_name].owner_declared_name,
) )
return rename_map return rename_map
def _record_provenance( def _record_provenance(
@ -2168,14 +2168,14 @@ class _BuildSessionLibrary(ILibrary):
renamed_from: str | None, renamed_from: str | None,
) -> None: ) -> None:
self._provenance[final_name] = CellProvenance( self._provenance[final_name] = CellProvenance(
final_name=final_name, final_name = final_name,
requested_name=requested_name, requested_name = requested_name,
kind=kind, kind = kind,
owner_declared_name=owner_declared_name, owner_declared_name = owner_declared_name,
emitted_via=emitted_via, emitted_via = emitted_via,
build_chain=build_chain, build_chain = build_chain,
renamed_from=renamed_from, renamed_from = renamed_from,
) )
if owner_declared_name is not None and final_name not in self._owned_cells[owner_declared_name]: if owner_declared_name is not None and final_name not in self._owned_cells[owner_declared_name]:
self._owned_cells[owner_declared_name].append(final_name) self._owned_cells[owner_declared_name].append(final_name)
@ -2226,7 +2226,7 @@ class _BuildSessionLibrary(ILibrary):
if self._overlay[name] is not pattern: if self._overlay[name] is not pattern:
raise BuildError( raise BuildError(
f'Recipe for "{name}" wrote a different pattern into the session under its own name.' f'Recipe for "{name}" wrote a different pattern into the session under its own name.'
) )
else: else:
self[name] = pattern self[name] = pattern
self._states[name] = 'built' self._states[name] = 'built'
@ -2260,17 +2260,17 @@ class _BuildSessionLibrary(ILibrary):
name: frozenset(self._dependency_graph.get(name, set())) name: frozenset(self._dependency_graph.get(name, set()))
for name in self._builder._declarations for name in self._builder._declarations
if name in self._dependency_graph or name in requested_roots if name in self._dependency_graph or name in requested_roots
} }
owned_cells = { owned_cells = {
name: tuple(cells) name: tuple(cells)
for name, cells in self._owned_cells.items() for name, cells in self._owned_cells.items()
} }
return BuildReport( return BuildReport(
requested_roots=tuple(dict.fromkeys(requested_roots)), requested_roots = tuple(dict.fromkeys(requested_roots)),
provenance=dict(self._provenance), provenance = dict(self._provenance),
owned_cells=owned_cells, owned_cells = owned_cells,
dependency_graph=dependency_graph, dependency_graph = dependency_graph,
) )
def to_overlay(self) -> ILibrary: def to_overlay(self) -> ILibrary:
return self._overlay return self._overlay