Skip to content

docs: Running agents section for Copilot Pro+ (sessions, streaming, state strategies)#650

Open
mick-gsk wants to merge 1 commit into
mainfrom
split/576-docs-running-agents-copilot
Open

docs: Running agents section for Copilot Pro+ (sessions, streaming, state strategies)#650
mick-gsk wants to merge 1 commit into
mainfrom
split/576-docs-running-agents-copilot

Conversation

@mick-gsk
Copy link
Copy Markdown
Owner

@mick-gsk mick-gsk commented May 3, 2026

Split from #576 (ADR-100 monorepo migration). Part of the PR decomposition into atomic, reviewable units.

Copilot AI review requested due to automatic review settings May 3, 2026 16:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mick-gsk
Copy link
Copy Markdown
Owner Author

mick-gsk commented May 3, 2026

Superseded by #720 (drift-output package migration, correctly titled) and #721 (ADR-100/harness docs). Original PR was mislabeled as 'docs: Running agents' but actually contained output package migration (43 files).

@mick-gsk mick-gsk closed this May 3, 2026
Comment on lines +1038 to +1039
"FALSE-FIX CHECK: if new upward imports appear elsewhere,"
" the violation was shifted not fixed",
Comment on lines +1049 to +1050
"FALSE-FIX CHECK: renaming without body change will not resolve this"
" — body hash must differ",
Comment on lines +1074 to +1075
"SIDE-EFFECT NOTE: repair commits may cause transient TVS findings"
" — this stabilizes over time",
Comment on lines +1123 to +1124
"FALSE-FIX CHECK: renaming without behaviour change may be valid if"
" the new name accurately describes the implementation",
"text": rule_rec.description,
"markdown": f"**{rule_rec.title}**: {rule_rec.description}",
}
except (ImportError, AttributeError, KeyError, TypeError):
if msg_rec:
combined = f"{base_text} | {msg_rec.title}"
base_text = combined[:400] if len(combined) > 400 else combined
except (ImportError, AttributeError, KeyError, TypeError):
meta = get_meta(signal_type)
if meta:
return meta.signal_name
except (ImportError, AttributeError, KeyError, TypeError):
rec = generate_recommendation(finding)
if rec:
return rec.title[:90]
except (ImportError, AttributeError, KeyError, TypeError):
minutes = int(duration // 60)
seconds = int(duration % 60)
console.print(f" Duration: {minutes}m {seconds}s")
except (TypeError, ValueError):
@github-actions github-actions Bot added documentation Improvements or additions to documentation output Changes to CLI output or formatters labels May 3, 2026
@mick-gsk mick-gsk reopened this May 4, 2026
@mick-gsk mick-gsk added the release:docs Include this PR under Documentation in release notes label May 4, 2026
@github-actions github-actions Bot added agent-review-requested Agent review was requested automatically lane/standard Fixes and features — standard review path lane/high-risk Signals/scoring/architecture — full gates + audit required labels May 4, 2026
@mick-gsk mick-gsk added the size/XL Diff ≥ 500 lines — consider splitting label May 4, 2026
@github-actions github-actions Bot added the has-conflicts PR has merge conflicts with the base branch label May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-review-requested Agent review was requested automatically documentation Improvements or additions to documentation has-conflicts PR has merge conflicts with the base branch lane/high-risk Signals/scoring/architecture — full gates + audit required lane/standard Fixes and features — standard review path output Changes to CLI output or formatters release:docs Include this PR under Documentation in release notes size/XL Diff ≥ 500 lines — consider splitting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants