Skip to content

test: Phase B architectural gate validation#55

Open
aneesh-spec wants to merge 17 commits intolangchain-ai:mainfrom
aneesh-spec:agent-test-phase-b
Open

test: Phase B architectural gate validation#55
aneesh-spec wants to merge 17 commits intolangchain-ai:mainfrom
aneesh-spec:agent-test-phase-b

Conversation

@aneesh-spec
Copy link
Copy Markdown

Purpose

Tests the Phase B architectural gate by simulating an agent fixing the same --urls colon-splitting bug that was fixed in aneesh-fix (creator patch), but using a different implementation approach.

Setup

Details
Creator patch aneesh-fix — inline fix with is_windows_drive check (12 LOC)
Agent patch This branch — fix via _is_label_prefix() helper function (180 LOC incl. tests)

Patches

  • .github/patches/creator.diffcli.py fix only from aneesh-fix (human reference)
  • .github/patches/agent.diffcli.py fix + F2P tests from this branch (agent output)

How to trigger Phase B gate

Go to Actions → Architecture Gate — Phase B → Run workflow with:

Input Value
agent_diff_path .github/patches/agent.diff
creator_diff_path .github/patches/creator.diff
base_sha main
head_sha agent-test-phase-b

Expected results

  • scope_score — should pass (agent touched same files as creator)
  • blast_ratio — may fail (agent wrote ~180 LOC vs creator's 12 LOC → ratio ≈ 0.07)
  • api_surface_score — should pass (no breaking changes)
  • new_dependencies_count — should pass (no new deps)
  • dead_code_count — should pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants