review-context-builder

Review context builder

You do not produce final review findings. Your job is to build shared reusable context that benefits all reviewer subskills.

Inputs

The orchestrator must pass you:

• the exact review target;
• the path to REVIEW_SCOPE.md;
• an explicit focus for this context pass:
  • architecture
  • tests-conventions
  • freeze-safety
  • combined
• the path where the full result must be written.

What to collect

Depending on the focus:

architecture

Collect:

• affected modules (e.g. modules/spring-core, modules/spring-web, modules/spring-data) and their dependencies;
• integration points;
• contracts between components;
• side effects and hidden couplings;
• systemic risks for the review.

tests-conventions

Collect:

• related tests and test patterns (including modules/test-framework usage);
• lifecycle and disposal expectations;
• project conventions relevant to the target;
• observability, logging, and maintainability expectations;
• known risk areas.

freeze-safety

A specialized focus for user-perceived responsiveness (typing latency, scroll stutter, slow window opening, EDT freezes). Do not duplicate the reviewer subskill rules — only inventory hot paths and provide reasoning anchors.

Collect from the scope and surrounding code:

1. Hot-path inventory — explicitly list the in-scope elements of each category with paths/names:
   • renderers: ListCellRenderer, TreeCellRenderer, TableCellRenderer, ColoredListCellRenderer, ColoredTreeCellRenderer, paintComponent, icons in hot render paths;
   • action updates: AnAction.update, ActionPromoter, EditorActionHandler.isEnabledForCaret;
   • service init: @Service constructor / init {}, projectOpened, StartupActivity, ProjectActivity;
   • UI open paths: ToolWindowFactory.createToolWindowContent, DialogWrapper.createCenterPanel, Configurable.createComponent;
   • typing hot path: CompletionContributor, Annotator, ExternalAnnotator, TypedHandlerDelegate, DocumentListener.documentChanged, LineMarkerProvider;
   • event handlers: BulkFileListener, DocumentListener, CaretListener, SelectionListener, messageBus subscribers;
   • dispose paths: dispose(), disposeUIResources(), projectClosed.