Back to skill

Security audit

团队每小时复盘

Security checks for vulnerabilities and agentic risk

Overview

The skill is a disclosed team workflow playbook, but it asks agents to create persistent memory, shared knowledge, scheduled reviews, external reminders, and session restarts without enough scoping or user control.

Install only in a controlled Chinese-speaking team environment after adding clear opt-in for scheduled jobs, limits on which agents and memory stores may be accessed, redaction rules for secrets and personal or business-sensitive data, human review before generated rules become active skills, and a visible way to list, expire, and disable all recurring reviews, backups, reminders, and session restarts.

Vulnerability Patterns
  • Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
  • Unauthorized Access and Privilege EscalationObtains permissions beyond the task's legitimate needs
  • System PersistenceInstalls backdoors, hooks, services, or scheduled tasks that survive the run
  • Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
  • Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
Findings (3)

T02 · Agent Memory Poisoning

Error
Location
task-knowledge-framework.md:7
Finding

Persistent Instruction Injection Through Shared Memory and Generated Skill Files

Content
View full analysis

Vulnerability Details

File Location: task-knowledge-framework.md, lines 7-18
Vulnerability Type: Persistent memory and instruction poisoning
Risk Level: High

Evidence

The following is an English translation of the complete relevant documentation segment:

markdown
### Task Persistence Iron Law
- Prohibited: Task loss. All important tasks must be written to MEMORY.md.
- Format: Task description + acceptance criteria + deadline + owner + status.
- Persistence: Before starting work, every agent must confirm the current tasks in MEMORY.md.
- Incorrect example: Context for the mobile remote-control deployment task was lost.
- Improvement: Establish a task knowledge base in which every task has a unique ID and complete record.

### Session Management Iron Law
- Prohibited: Memory failure. Every agent must have a daily memory file.
- Inspection: The steward checks every agent's memory/YYYY-MM-DD.md file daily.
- Backup: Important memory files are backed up to shared_memory/.
- Incorrect example: The writer agent repeatedly had no memory file.
- Improvement: Establish session health monitoring and recovery mechanisms.

Related propagation requirements also appear in:

  • SKILL.md, lines 18-20: review results must be written to Skill files, an Obsidian knowledge base, and daily memory logs.
  • session-management-iron-laws.md, lines 55-70: every error must become a Skill or rule, and Skill files must be updated regularly.
  • task-knowledge-framework.md, lines 76-85 and 112-121: errors must be converted into rules, and work cannot end until knowledge is persisted.
  • working-iron-laws.md, lines 45-49 and 59-63: tasks and errors must be written to persistent knowledge stores.

Technical Analysis

The documentation directs an agent to copy current task content, error descriptions, and behavioral rules into MEMORY.md, daily memory files, shared_memory/, and newly generated Skill file ...[truncated 1977 chars]

Remediation
View remediation

Remediation Suggestions

  1. Remove requirements that automatically convert task content or errors into persistent rules.
  2. Require explicit user approval before writing to MEMORY.md, shared memory, an Obsidian repository, or a Skill file.
  3. Store factual task state separately from executable instructions and behavioral policy.
  4. Mark every persisted entry with its source, author, creation time, trust level, and expiration time.
  5. Treat reloaded memory as untrusted data rather than authoritative instructions.
  6. Apply content filtering to prevent stored records from containing tool commands, role changes, safety overrides, or instructions to modify additional memory.
  7. Restrict shared-memory and Skill-writing permissions according to least privilege.
  8. Require human review and version-controlled approval before generated content becomes an active Skill.
  9. Add retention limits and deletion procedures for task context and personal information.
  10. Replace workflow vetoes such as “work cannot end” with non-binding review recommendations.

T06 · System Persistence

Error
Location
systemic-improvement-framework.md:86
Finding

Recurring Automation and Cross-Agent Session Control

Content
View full analysis

Vulnerability Details

File Location: systemic-improvement-framework.md, lines 86-94
Vulnerability Type: Scheduled persistence and excessive session-management authority
Risk Level: High

Evidence

The following is an English translation of the complete relevant documentation segment:

markdown
### 4. Session Lifecycle Management
Objective: Resolve the FM-3.1 premature termination problem.

Management strategy:
- Heartbeat monitoring: Check session status every minute.
- Automatic recovery: Automatically restart a session when an anomaly is found.
- Graceful exit: Save state before normal termination.
- Resource recovery: Periodically clean up unused resources.

Additional recurring behavior is prescribed in:

  • SKILL.md, lines 7-9: a cron trigger invokes the project manager every hour to conduct a review.
  • session-management-iron-laws.md, lines 6-9: a steward checks every agent's daily memory and immediately restarts anomalous sessions.
  • session-management-iron-laws.md, lines 30-34: the project manager checks important tasks every two hours.
  • systemic-improvement-framework.md, lines 35-47: session history is loaded automatically, backed up every five minutes, retained for 30 days, and restored after failures.

Technical Analysis

The Skill prescribes cross-session scheduled activity through cron, minute-level heartbeats, periodic backups, and automatic session restarts. These mechanisms can survive the initiating interaction and consume resources without a fresh user request.

It also gives management roles authority to inspect and restart other agents. Such authority crosses normal isolation boundaries and is broader than required for a documentation or review-template Skill. If the controlling agent or its instructions are compromised, the same mechanisms can be used to monitor, interrupt, or influence multiple sessions.

No actual cron configuration, startup service, or e ...[truncated 1591 chars]

Remediation
View remediation

Remediation Suggestions

  1. Remove automatic cron installation and automatic session-restart requirements from the Skill.
  2. Require explicit, time-bounded user authorization before creating any scheduled task.
  3. Display the schedule, purpose, owner, expiration time, and expected resource usage before activation.
  4. Default monitoring to read-only health signals that do not expose conversation or memory content.
  5. Require manual confirmation before restarting another agent's session.
  6. Scope session-management permissions to specifically authorized agent identifiers.
  7. Add rate limits, restart backoff, maximum retry counts, and circuit breakers.
  8. Provide a visible mechanism to enumerate and disable all schedules created by the Skill.
  9. Record immutable audit events for memory access, session inspection, backup, restoration, and restart operations.
  10. Ensure all schedules expire automatically when the associated task or user session ends.

T05 · Unauthorized Access and Privilege Escalation

Warning
Location
decision-management-framework.md:112
Finding

Unrestricted Collection and Multi-Channel Distribution of Operational Information

Content
View full analysis

Vulnerability Details

File Location: decision-management-framework.md, lines 112-120
Vulnerability Type: Excessive collection and dissemination of internal information
Risk Level: Medium

Evidence

The following is an English translation of the complete relevant documentation segment:

markdown
### Multi-channel reminders
- Feishu messages: Formal reminders containing complete information.
- Heartbeat checks: Include decision status in daily reports.
- Knowledge-base updates: Mark decision progress in MEMORY.md.
- A2A communication: Send reminder information through a dedicated session.

Related collection requirements include:

  • SKILL.md, lines 12-15: reviews collect errors, task completion status, owner feedback, expectations, and improvement measures.
  • decision-management-framework.md, lines 73-79: decision records include detailed background, affected projects, impact, urgency, reminder history, requested information, risk analysis, and contacts.
  • session-management-iron-laws.md, lines 42-46: agents are directed to inspect model API-call records to monitor usage limits.

Technical Analysis

The framework collects detailed operational information and distributes it through several destinations: Feishu, system or heartbeat reports, MEMORY.md, knowledge stores, and inter-agent sessions. The instructions do not define field-level access controls, recipient validation, redaction, consent, data classification, or retention boundaries.

Sending “complete information” through every reminder channel violates data-minimization principles. A recipient that legitimately needs a decision identifier and deadline may not need detailed project context, personnel feedback, risk analysis, contact information, or API-usage records.

The audited files do not contain credentials, hard-coded remote endpoints, or executable exfiltration logic. The risk occurs if an agent with access to internal reco ...[truncated 1332 chars]

Remediation
View remediation

Remediation Suggestions

  1. Replace “complete information” with a minimum reminder schema containing only the decision ID, summary, deadline, status, and authorized contact.
  2. Classify data before storage or transmission and prohibit sensitive fields from general reminder channels.
  3. Validate recipients and channel authorization before every transmission.
  4. Require explicit consent before transmitting personal feedback, contact details, or detailed task failures.
  5. Redact secrets, identifiers, private feedback, and unnecessary operational metadata.
  6. Use separate access-controlled stores for decision details and reminder summaries.
  7. Encrypt sensitive records in transit and at rest.
  8. Define retention periods and automatically delete expired reminder histories and API-usage metadata.
  9. Log who accessed or transmitted each record without duplicating the sensitive payload in logs.
  10. Add a human approval step before information is copied from internal memory to Feishu or another externally accessible channel.
Vulnerability Patterns
  • Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
  • Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
  • Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
  • Supply ChainUnpinned Dependencies, External Script Fetching, Obfuscated Code
  • Excessive AgencyUnrestricted Tool Access, Autonomous Decision Making, Scope Creep
Findings (9)

Natural-Language Policy Violations

Medium
Category
Not specified by scanner
Confidence
89% confidence
Finding

The file’s operational instructions and templates are entirely in Chinese, which imposes a specific language on users and operators. Under the policy, forcing a language without opt-in or justification is a natural-language policy violation.

Content

No source excerpt is available for this finding.

Natural-Language Policy Violations

Medium
Category
Not specified by scanner
Confidence
93% confidence
Finding

This markdown file contains user-facing operational instructions exclusively in Chinese, with no indication that users can choose another language or that the locale restriction is intentional and justified. Under the policy, forcing a specific language without user opt-in is a natural-language policy violation.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
92% confidence
Finding

The skill requires persistent storage of daily memory files and important task details, but it provides no data-classification, minimization, retention, or secret-handling guidance. In practice, this encourages operators to store sensitive operational context, credentials, personal data, or confidential business information in durable plaintext artifacts that can be broadly accessed or retained longer than necessary.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
89% confidence
Finding

The A2A guidance focuses on routing discipline and prior leakage incidents, but it does not define confidentiality controls for message content itself. That omission can lead users to believe that using the correct session format is sufficient, even when sensitive data is being forwarded, copied, or exposed across agent channels without sanitization or authorization checks.

Content

No source excerpt is available for this finding.

Natural-Language Policy Violations

Medium
Category
Not specified by scanner
Confidence
88% confidence
Finding

The skill content is written entirely in Chinese and does not indicate that language selection is optional or limited to a justified region-specific context. Per the policy, forcing a specific language without user opt-in is a natural-language policy violation.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
94% confidence
Finding

The skill requires that all important tasks be written to a persistent MEMORY.md file, but it provides no safeguards about excluding secrets, personal data, credentials, or other sensitive project information. In an agent workflow, this can normalize over-collection and long-term retention of sensitive data, increasing the risk of unintended disclosure, cross-task leakage, or later misuse.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
95% confidence
Finding

The instruction to back up important memory files into shared_memory/ expands the access scope of persisted data without any notice about broader sharing, retention, or access controls. If MEMORY.md contains sensitive operational details, this replication can amplify exposure across agents, sessions, or users and make accidental disclosure more likely.

Content

No source excerpt is available for this finding.

Natural-Language Policy Violations

Medium
Category
Not specified by scanner
Confidence
92% confidence
Finding

The file is entirely framed as mandatory operational rules in Chinese, including requirements that 'all instructions must be concise and clear' for workers, but it provides no option for another language or opt-in to a Chinese-only workflow. Under the policy rule, forcing a specific language without user choice is a natural-language policy concern.

Content

No source excerpt is available for this finding.

Natural-Language Policy Violations

Low
Category
Not specified by scanner
Confidence
81% confidence
Finding

All user-facing instructions in this file are presented exclusively in Chinese, and there is no indication that users can choose another language or that the skill is intentionally limited to a Chinese-speaking context. That can create a language policy concern if the organization expects language neutrality or explicit user opt-in for locale constraints.

Content

No source excerpt is available for this finding.

Static analysis

No suspicious patterns detected.