forked from fullsend-ai/fullsend
-
Notifications
You must be signed in to change notification settings - Fork 0
perf(#2354): bound enrollment wait with timeout and backoff #76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
guyoron1
wants to merge
53
commits into
main
Choose a base branch
from
mirror/2359-2354-enrollment-wait-timeout
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
63c27e4
docs: add design spec for triage prerequisites action (#401)
ralphbean ba99ae3
docs: add implementation plan for triage prerequisites action (#401)
ralphbean 9a35c91
feat(config): add create_issues allowlist config (#401)
ralphbean d4a394e
refactor: update NewOrgConfig/NewPerRepoConfig callers for create_iss…
ralphbean e492ac7
feat(schema): replace blocked with prerequisites action (#401)
ralphbean b2055cb
feat(triage): replace blocked action with prerequisites in agent prom…
ralphbean c48a832
docs: document prerequisites action and create_issues config (#401)
ralphbean 3a44b0c
feat(triage): handle prerequisites action in post-script (#401)
ralphbean 6f79d87
fix(triage): correct label name in agent prompt and remove dead code …
ralphbean 080368c
fix(triage): update post-triage tests for prerequisites action (#401)
ralphbean 11bae49
fix(triage): update schema validation tests for prerequisites action …
ralphbean e57f10a
fix(triage): address review feedback on prerequisites action (#401)
ralphbean 2e040b5
chore(skills): add e2e-health skill
ralphbean 7c40a70
fix(skills): escape example link in e2e-health SKILL.md
ralphbean 162dce2
fix(skills): address review feedback on e2e-health skill
ralphbean 80a414d
fix: widen CSMA jitter after rate-limit reset to prevent thundering herd
ralphbean 61f467d
test: add Phase 2 integration tests for ADR-0045 forge-portable harne…
ggallen ded059b
fix(#2130): mint fresh tokens for status comments on demand
ggallen 3c9f0db
Merge pull request #2304 from fullsend-ai/fix/csma-jitter-window
ralphbean 7249b34
fix(skills): remove markdown link syntax from e2e-health example table
ralphbean df020f5
Merge pull request #2301 from fullsend-ai/add-e2e-health-skill
ralphbean 3ae6f72
fix(#2343): add post-reset spread to _github_csma_sleep_after_rate_limit
fullsend-ai-coder[bot] 966abbf
Merge pull request #2344 from fullsend-ai/agent/2343-csma-spread-rate…
ralphbean d988d32
merge: resolve conflict with main in admin.go
ralphbean a24ffd1
style: gofmt config.go after merge
ralphbean d6988a9
Merge pull request #2299 from ggallen/worktree-fix-2130-status-token
ggallen 515e49b
Merge pull request #2321 from ggallen/worktree-phase2-pr6
ggallen 133ed6e
Merge pull request #2197 from fullsend-ai/rbean/401-triage-decompose-…
ralphbean 8526637
perf(#2354): bound enrollment wait with timeout and backoff
fullsend-ai-coder[bot] 2b8f453
Add QualityFlow output for GH-2354 [skip ci]
4451fdb
Add STP output for GH-2354 [skip ci]
ac52410
Add QualityFlow STP review for GH-2354 [skip ci]
b031e37
Add QualityFlow output for GH-2354 [skip ci]
52b91fd
Add QualityFlow STD review for GH-2354 [skip ci]
797f403
Add test output for GH-2354 [skip ci]
002783c
Add QualityFlow tests for GH-2354
guyoron1 a24ac7b
Add QualityFlow output for GH-76 [skip ci]
aebc721
Add STP output for GH-76 [skip ci]
db0e6f8
Add QualityFlow STP review output for GH-76 [skip ci]
46a3276
Refine STP for GH-76: APPROVED_WITH_FINDINGS → APPROVED [skip ci]
6cd342e
Add QualityFlow STD review output for GH-76 [skip ci]
b1ed45b
Add QualityFlow STD refiner output for GH-76 [skip ci]
94ed7e5
Add QualityFlow tests for GH-76 [skip ci]
1d8e7ae
Clean QualityFlow artifacts for GH-76
guyoron1 9a7e2b3
Add QualityFlow output for GH-76 [skip ci]
8d2f147
Add STP output for GH-76 [skip ci]
118b076
Add QualityFlow STP review for GH-76 [skip ci]
72232c6
Add QualityFlow output for GH-76 [skip ci]
fa5b4a0
Add QualityFlow STD review for GH-76 [skip ci]
a5882da
Add QualityFlow output for GH-76 [skip ci]
8575f66
Add QualityFlow tests for GH-76 [skip ci]
4055f4d
Clean QualityFlow artifacts for GH-76
guyoron1 df0bcd9
chore: remove old qf-tests/ artifacts
guyoron1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[medium] fail-open
The reconciliation step is gated on inputs.mint-url being non-empty. When mint-url is not provided, the entire status reconciliation is silently skipped with no warning.
Suggested fix: Add a warning annotation when reconciliation is skipped due to missing mint-url.