Smitaambiger commented on issue #407: URL: https://github.com/apache/burr/issues/407#issuecomment-3873347103
Hi @elijahbenizzy I’d like to work on this issue if it’s still open. Based on a quick pass through the function-based action implementation, this seems feasible via a scoped AST check: - Target `FunctionBasedAction` / `FunctionBasedStreamingAction` - Parse the source of the originating function - Detect `state["<key>"]` accesses with string literals - Compare against the declared `reads=[...]` - Fail fast when undeclared reads are detected I’d plan to run this validation at application build time (not decorator import time) and scope it strictly to the assumptions listed in the issue. Before starting, can you confirm: 1. Build-time validation is the right hook? 2. Hard error vs warning for the initial behavior? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
