Skip to content

Issue #95: Prompt Refinement Tracking#97

Open
minouris wants to merge 2 commits into
mainfrom
feature/issue-95-prompt-refinement
Open

Issue #95: Prompt Refinement Tracking#97
minouris wants to merge 2 commits into
mainfrom
feature/issue-95-prompt-refinement

Conversation

@minouris

Copy link
Copy Markdown
Owner

Addresses #95

This PR tracks the implementation of prompt refinement improvements identified during Issue #63.

Implementation Sequence

Per Issue #95, this work is broken down into sub-issues to be resolved in order:

  1. Proposal: Split feature plan documents into multiple focused files #93 - Multi-file plan structure
  2. Category 2: Processing Capacity - Multi-file plan structure refinements #96 - File-specific template refinements
  3. Context overflow from instruction file system during planning workflows #68 - Context overflow handling
  4. Improve Step 8 prompt clarity and workflow tracking #83 - Step 8 instruction clarity
  5. Category 1 solutions (6 prompt refinements)
  6. Category 3 monitoring (Workflow Execution Policy)

Current Status

Branch: feature/issue-95-prompt-refinement
Sub-branches: Each sub-issue will have its own feature branch merged into this one

Changes in This PR

  • File Editing Policy added to .github/copilot-instructions.md
  • .tmp/ directory added to .gitignore

Further changes will be added as sub-issues are completed.

Copilot AI review requested due to automatic review settings December 29, 2025 00:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR establishes foundational policies for prompt refinement tracking as part of Issue #95. It adds critical file editing policies and markdown formatting guidelines to prevent issues identified during Issue #63 implementation.

Key Changes:

  • Added File Editing Policy to enforce proper tool usage and workspace-local temporary files
  • Added Markdown Formatting guidelines for handling nested code blocks
  • Added .tmp/ directory to .gitignore for workspace-local temporary files

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
.gitignore Adds .tmp/ directory exclusion to support workspace-local temporary file policy
.github/copilot-instructions.md Adds comprehensive File Editing Policy (lines 55-148) and Markdown Formatting guidelines (lines 270-277) to enforce best practices for file operations and documentation

When creating content with nested fenced code blocks:
- **Simple nesting:** Use quad-backticks (````) for outer blocks that contain triple-backticked code (per CommonMark spec)
- **Complex nesting (lists + code blocks):** Move code blocks outside list indentation when inside markdown blocks
- ❌ WRONG: List item with indented ```code``` block inside ````markdown block

Copilot AI Dec 29, 2025

Copy link

Choose a reason for hiding this comment

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

The backtick rendering in line 274 may not display correctly in markdown. The text reads "List item with indented code block inside ````markdown block" but the three backticks in the middle will likely be interpreted as a code fence opening, breaking the rendering. Consider using literal backtick references or escaping them differently, such as "List item with indented triple-backticked code block inside quad-backticked markdown block" or using backtick entities.

Suggested change
- ❌ WRONG: List item with indented ```code``` block inside ````markdown block
- ❌ WRONG: List item with indented triple-backticked code block inside quad-backticked markdown block

Copilot uses AI. Check for mistakes.
Comment on lines +274 to +275
- ❌ WRONG: List item with indented ```code``` block inside ````markdown block
- ✅ CORRECT: List items, then standalone ```code``` block (not indented), then more list items

Copilot AI Dec 29, 2025

Copy link

Choose a reason for hiding this comment

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

The backtick rendering in line 275 may not display correctly in markdown. The text reads "List items, then standalone code block (not indented)" but the three backticks will likely be interpreted as a code fence opening, breaking the rendering. Consider using literal backtick references or escaping them differently, such as "List items, then standalone triple-backticked code block (not indented)" or using backtick entities.

Suggested change
- ❌ WRONG: List item with indented ```code``` block inside ````markdown block
- ✅ CORRECT: List items, then standalone ```code``` block (not indented), then more list items
- ❌ WRONG: List item with indented triple-backticked `code` block inside ````markdown block
- ✅ CORRECT: List items, then standalone triple-backticked `code` block (not indented), then more list items

Copilot uses AI. Check for mistakes.
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