Illegal catshark#323912
Open
NguyenCuong1989 wants to merge 10 commits into
Open
Conversation
This workflow installs Deno and runs linting and tests.
Cần phải tập trung sửa các lỗi tồn đọng đảm bảo các repo hoạt động hoàn hảo
commit
Author
|
@microsoft-github-policy-service agree |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces a set of AI/agent workflow scaffolding files (chatmodes + “memory bank” markdown docs), updates workspace configuration under .vscode/, and adds new GitHub Actions workflows plus several local/generated-tooling artifacts.
Changes:
- Add new
.github/*.chatmode.mdmode definitions and amemory-bank/documentation set. - Modify VS Code workspace config (
.vscode/settings.json,.vscode/mcp.json,.vscode/launch.json). - Add new CI workflows (
deno.yml,datadog-synthetics.yml) and commit snapshot/local-tool directories (.snapshots/,.lh/,.local-versions/).
Reviewed changes
Copilot reviewed 23 out of 25 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| memory-bank/systemPatterns.md | Adds placeholder “system patterns” documentation. |
| memory-bank/projectBrief.md | Adds placeholder project brief content. |
| memory-bank/progress.md | Adds placeholder progress tracking doc. |
| memory-bank/productContext.md | Adds placeholder product context doc. |
| memory-bank/decisionLog.md | Adds decision log table header. |
| memory-bank/architect.md | Adds “MemoriPilot” architecture notes (placeholder). |
| memory-bank/activeContext.md | Adds placeholder active context doc. |
| .vscode/settings.json | Expands file excludes and changes chat terminal output location; adds IIS setting. |
| .vscode/mcp.json | Replaces MCP server config with an external uvx-based server. |
| .vscode/launch.json | Adds many blank lines in configurations array. |
| .snapshots/sponsors.md | Adds third-party promotional content. |
| .snapshots/readme.md | Documents snapshots directory usage. |
| .snapshots/config.json | Adds configuration for a “Snapshots for AI” tool. |
| .local-versions/.gitignore | Adds ignore rules for a local-only directory. |
| .lh/.lhignore | Adds ignore config for a local-history extension. |
| .gitignore | Adds an ignore entry for a .github/instructions/ file (with Windows path separators). |
| .github/workflows/deno.yml | Adds a Deno lint/test workflow. |
| .github/workflows/datadog-synthetics.yml | Adds a Datadog synthetics workflow requiring secrets. |
| .github/ISSUE_TEMPLATE/copilot_bug_report.md | Removes default label assignment. |
| .github/instructions/codacy.instructions.md | Adds Codacy MCP-focused agent instructions. |
| .github/debug.chatmode.md | Adds a “Debug Expert” chat mode definition. |
| .github/code.chatmode.md | Adds a “Code Expert” chat mode definition. |
| .github/ask.chatmode.md | Adds a “Project Assistant” chat mode definition. |
| .github/architect.chatmode.md | Adds a “System Architect” chat mode definition. |
Comments suppressed due to low confidence (1)
.vscode/settings.json:220
iis.configDiris a workspace-specific setting for an IIS extension and isn’t referenced anywhere else in the repo. Keeping it in the shared.vscode/settings.jsonwill affect all contributors’ workspace settings without an apparent project need.
// --- Workbench ---
"editor.aiStats.enabled": true, // Team selfhosting on ai stats
"azureMcp.enabledServices": [
Comment on lines
1
to
11
| { | ||
| "servers": { | ||
| "vscode-automation-mcp": { | ||
| "microsoft/markitdown": { | ||
| "type": "stdio", | ||
| "command": "npm", | ||
| "command": "uvx", | ||
| "args": [ | ||
| "run", | ||
| "start-stdio" | ||
| "markitdown-mcp@0.0.1a4" | ||
| ], | ||
| "cwd": "${workspaceFolder}/test/mcp" | ||
| }, | ||
| "component-explorer": { | ||
| "type": "stdio", | ||
| "command": "npm", | ||
| "args": [ | ||
| "exec", | ||
| "--no", | ||
| "--", | ||
| "component-explorer", | ||
| "mcp", | ||
| "-p", | ||
| "./test/componentFixtures/component-explorer.json", | ||
| "--use-daemon", | ||
| "-vv" | ||
| ] | ||
| "gallery": "https://api.mcp.github.com", | ||
| "version": "1.0.0" | ||
| } |
Comment on lines
3
to
19
| "configurations": [ | ||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
| { |
Comment on lines
+16
to
+25
| on: | ||
| push: | ||
| branches: [ "main" ] | ||
| pull_request: | ||
| branches: [ "main" ] | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
|
|
Comment on lines
+1
to
+16
| # This workflow uses actions that are not certified by GitHub. | ||
| # They are provided by a third-party and are governed by | ||
| # separate terms of service, privacy policy, and support | ||
| # documentation. | ||
|
|
||
| # This workflow will install Deno then run `deno lint` and `deno test`. | ||
| # For more information see: https://github.com/denoland/setup-deno | ||
|
|
||
| name: Deno | ||
|
|
||
| on: | ||
| push: | ||
| branches: ["main"] | ||
| pull_request: | ||
| branches: ["main"] | ||
|
|
Comment on lines
+1
to
+12
| { | ||
| "excluded_patterns": [ | ||
| ".git", | ||
| ".gitignore", | ||
| "gradle", | ||
| "gradlew", | ||
| "gradlew.*", | ||
| "node_modules", | ||
| ".snapshots", | ||
| ".idea", | ||
| ".vscode", | ||
| "*.log", |
Comment on lines
+1
to
+16
| --- | ||
| # Codacy Rules | ||
| Configuration for AI behavior when interacting with Codacy's MCP Server | ||
|
|
||
| ## CRITICAL: After ANY successful `edit_file` or `reapply` operation | ||
| - YOU MUST IMMEDIATELY run the `codacy_cli_analyze` tool from Codacy's MCP Server for each file that was edited, with: | ||
| - `rootPath`: set to the workspace path | ||
| - `file`: set to the path of the edited file | ||
| - `tool`: leave empty or unset | ||
| - If any issues are found in the new edits, propose and apply fixes for them. | ||
| - > NOTE: Failure to follow this rule is considered a critical error. | ||
|
|
||
| ## When you tried to run the `codacy_cli_analyze` tool and the Codacy CLI is not installed | ||
| - Ask the user 'Codacy CLI is not installed. Would you like me to install it now?' | ||
| - If the user responds with "yes", run the `codacy_cli_install` tool and then continue with the original task | ||
| - If the user responds with "no", instruct the user that they can disable automatic analysis in the extension settings |
Comment on lines
+1
to
+6
| # list file to not track by the local-history extension. comment line starts with a '#' character | ||
| # each line describe a regular expression pattern (search for 'Javascript regex') | ||
| # it will relate to the workspace directory root. for example: | ||
| # '.*\.txt' ignores any file with 'txt' extension | ||
| # '/test/.*' ignores all the files under the 'test' directory | ||
| # '.*/test/.*' ignores all the files under any 'test' directory (even under sub-folders) |
Comment on lines
+1
to
+2
| * | ||
| !.gitignore |
Comment on lines
+1
to
+8
| --- | ||
| description: Design robust and scalable software systems, make high-level architectural decisions, and maintain the project's memory bank. | ||
| tools: ['changes', 'codebase', 'editFiles', 'extensions', 'fetch', 'findTestFiles', 'githubRepo', 'new', 'openSimpleBrowser', 'problems', 'runCommands', 'runNotebooks', 'runTasks', 'search', 'searchResults', 'terminalLastCommand', 'terminalSelection', 'testFailure', 'usages', 'vscodeAPI', 'logDecision', 'showMemory', 'switchMode', 'updateContext', 'updateMemoryBank', 'updateProgress'] | ||
| version: "1.0.0" | ||
| --- | ||
| # System Architect | ||
|
|
||
| You are an expert system architect in this workspace. Your goal is to help design robust and scalable software systems, make high-level architectural decisions, and maintain the project's memory bank. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.