Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

astep() passes unprocessed inputs to pre_run_step hooks

オープン 初心者向け
  • 何か問題がありますか？
#979 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 6 日以内に返信

@sivakumar-mahalingam がすでに取り組んでいます。

2026年10月18日 から。

  • #980 @sivakumar-mahalingam による — オープン

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
78/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
backend

調査の方向性

同期版の _step() と非同期版の _astep() の実装を読み、_process_inputs() が pre_run_step に対してどこで実行されるかに注目してください。処理済み入力と必須入力の不足に関するテストを、issue の例を参考に追加または適宜修正してください。両方の経路で、hook に action と同じ処理済み入力が渡され、検証に失敗した場合は hook がスキップされれば完了です。

索引モデルが issue の本文から書いたものです。

説明

status/needs-triage

Application.astep() passes raw, unprocessed inputs to pre_run_step lifecycle hooks, while Application.step() passes the validated and processed inputs that are actually provided to the action. This causes inconsistent lifecycle and tracking behavior between synchronous and asynchronous execution.

Current behavior

Application.step() calls _process_inputs() before invoking the pre_run_step hook. Consequently, undeclared inputs are removed and required inputs are validated before the hook runs.

Application.astep() invokes pre_run_step with the original input dictionary before calling _process_inputs().

For an action that declares no inputs:

step hook inputs:  {}
astep hook inputs: {"unused": 123}

There is also inconsistent behavior when a required input is missing:

pre_run_step calls:
step:  0
astep: 1

This means lifecycle adapters and tracking clients can record inputs that were never passed to the action.

Stack Traces

Not applicable. The inconsistency occurs without an exception when undeclared inputs are supplied.

Screenshots

Not applicable.

Steps to replicate behavior

  1. Run the following script:
import asyncio

from burr.core import ApplicationBuilder, State, action
from burr.lifecycle import PreRunStepHook


class CaptureInputs(PreRunStepHook):
    def __init__(self):
        self.inputs = []

    def pre_run_step(self, *, inputs, **kwargs):
        self.inputs.append(inputs)


@action(reads=[], writes=[])
def noop(state: State) -> State:
    return state


sync_hook = CaptureInputs()
sync_app = (
    ApplicationBuilder()
    .with_actions(noop)
    .with_entrypoint("noop")
    .with_hooks(sync_hook)
    .build()
)
sync_app.step(inputs={"unused": 123})


async_hook = CaptureInputs()
async_app = (
    ApplicationBuilder()
    .with_actions(noop)
    .with_entrypoint("noop")
    .with_hooks(async_hook)
    .build()
)
asyncio.run(async_app.astep(inputs={"unused": 123}))

print("step hook inputs:", sync_hook.inputs)
print("astep hook inputs:", async_hook.inputs)
  1. Observe the output:
step hook inputs: [{}]
astep hook inputs: [{"unused": 123}]

Library & System Information

  • Apache Burr: current main (9088aad8cb6de62b9923816b89eaa562c1162c65)
  • Python 3
  • Reproduced on Linux

Expected behavior

step() and astep() should have consistent lifecycle semantics:

  • Inputs should be validated and processed before pre_run_step is invoked.
  • The hook should receive the processed inputs that will be passed to the action.
  • The same processed input dictionary should be used by both the hook and the action.
  • pre_run_step should not be invoked when input validation fails.

Additional context

In _step(), _process_inputs() is called before pre_run_step, and the resulting action_inputs are passed to both the hook and the action.

In _astep(), pre_run_step currently receives the original inputs dictionary, and _process_inputs() is called later during action execution.

Related issue #36 added action inputs to lifecycle hooks and tracking, but it does not cover this synchronous/asynchronous inconsistency.

主要言語
Python
スター
2.6k
フォーク
209
平均マージ
10日 9時間
マージ済み PR（30日）
17

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

apache/burr のほかの issue

apache/burr の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。