Skip to content

chore: update fabric cicd version#187

Merged
ayeshurun merged 220 commits intomicrosoft:mainfrom
ayeshurun:update-fabric-cicd-version
Mar 12, 2026
Merged

chore: update fabric cicd version#187
ayeshurun merged 220 commits intomicrosoft:mainfrom
ayeshurun:update-fabric-cicd-version

Conversation

@ayeshurun
Copy link
Copy Markdown
Collaborator

No description provided.

ayeshurun and others added 30 commits September 17, 2025 17:11
chore: Introduce release workflowc
Alon Yeshurun and others added 25 commits February 19, 2026 14:34
When AI asks for author info, skip team notification. Team is only
notified after author responds and re-triage still can't resolve.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Move ai:* label cleanup before the if/else branch so it runs
unconditionally. Prevents stale labels like ai:question-triage:needs
maintainer input from remaining on issues.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add 'Needs Author Feedback' assessment category to all 3 prompts
- Rename 'Missing Details' to 'Needs Author Feedback' in bug-triage
- Add new category to question-triage and feature-triage
- Remove redundant response text fallback and needsHumanReview override
- Update workflow label detection to match new category name

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Merge Needs Maintainer Input and Needs Team Review into single
  Needs Team Review category in question-triage prompt
- Remove duplicate needs author feedback label from post-processing
  (action's ai:* label is sufficient)
- Remove ai:* label cleanup (keep action's assessment labels)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Extract CLI commands, flags, item types, auth methods, and error
codes from actual source files. Inject into system prompt (not issue
body) so AI backs up answers with real codebase data. Comments and
issue body stay clean.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
All AI triage comments now end with a footer instructing users
to tag @microsoft/fabric-cli-dev if the issue needs team attention
but wasn't escalated by the AI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Approach A (Action): github/ai-assessment-comment-labeler with codebase context injection
- Approach B (Code Agent): Custom scripts/ai_triage.py that greps source code and injects into prompt
- Both trigger on 'needs triage' label and post tagged comments (🅰️/🅱️)
- Both include collapsible label decision tables in comments
- Labels suppressed on both for fair testing
- Added CLI doc page URLs to all 3 prompt files

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- A: Use yaml.safe_load/dump to avoid YAML alias errors from markdown **bold**
- A: Remove bold markdown from injected context text
- B: Reduce code context limits (30K total, 4K/file, 8 files max)
- B: Add API error details to stderr for debugging

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Bug: Potential Bug and Needs Team Review → Escalated to team
- Question: Tied each category explicitly to its footer
- Feature: Tied each category explicitly to its footer
- Removed stale Codebase Reference instruction from feature prompt

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove numbered dimension checklists from all 3 prompts
- Instruct model to skip adequate aspects, highlight only missing/wrong/notable
- Reduce target length from 3-5 to 2-4 paragraphs
- Trim verbose preamble in ai_triage.py code context header

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Valuable Enhancement → 📋 Backlog candidate (no escalation)
- Help Wanted → 🤝 Community contribution welcome
- Out of Scope → ✅ No action needed
- Add 'needs discussion' to needsHumanReview in both workflows

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add tailored PEP/RFC/convention references to each prompt:
- Bug: packaging, CLI conventions, HTTP/REST, auth, file I/O, runtime
- Question: packaging, CLI conventions, HTTP/REST, auth, data formats
- Feature: CLI conventions, packaging, HTTP/REST, backward compat, auth

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…mpts

- Remove Approach B (agent workflow + ai_triage.py), archived on branch
- Move codebase reference from pipeline build step into prompt files
- Prompts now self-contained: commands, flags, item types, auth, hidden entities
- Remove /tmp/prompts indirection, use .github/prompts directly
- Clean up A/B testing artifacts (tagged comments, approach labels)
- Restore phase control flags (SUPPRESS_LABELS + SUPPRESS_COMMENTS)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…inks

Reference https://microsoft.github.io/fabric-cli/essentials/resource_types/
instead of static lists, keeping prompts always up to date.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ayeshurun ayeshurun requested a review from a team as a code owner March 12, 2026 05:34
@ayeshurun ayeshurun merged commit da4368b into microsoft:main Mar 12, 2026
12 of 13 checks passed
@ayeshurun ayeshurun deleted the update-fabric-cicd-version branch March 12, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants