Conversation
- Add anti-repetition mandates to reply prompts - Increase recent posts context from 8 to 12 for better pattern avoidance - Soften aggressive selling instructions in style guidelines - Add RELAXED MODE for more natural conversations - Write Oct 17 diary entry documenting the week's evolution
|
Caution Review failedThe pull request is closed. WalkthroughThe changes comprise documentation additions and prompt refinements across multiple components. A reflective diary entry documents design considerations around content intelligence and interaction patterns. Configuration updates increase recent post retention from 8 to 12 items and introduce variety directives to prompt builders, while character style prompts shift from prescriptive to contextual language. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Rationale: Four files touched across three distinct concerns (documentation, plugin configuration, character styling). While individual changes are primarily additive or cosmetic string updates without control flow modifications, the distributed nature of prompt refinements across multiple functions and the thematic coherence across different files warrant closer attention to ensure consistency in intent. Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull Request Overview
This PR softens aggressive “selling” behavior, adds anti-repetition guidance, and expands recent-post context to improve variety and reduce repetitive coordinate suggestions.
- Add anti-repetition and “relaxed mode” directives to style and prompt builders
- Increase recent posts context window from 8 to 12 to improve pattern awareness
- Add an Oct 17 diary documenting the evolution
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/character/style.ts | Updates style guidelines to discourage forced coordinate suggestions, add anti-repetition and relaxed mode language. |
| plugin-nostr/lib/text.js | Injects “VARIETY MANDATE” instructions into post/thread and DM prompt builders to reduce repetition. |
| plugin-nostr/lib/service.js | Increases recentAgentPosts window from 8 to 12 across multiple flows. |
| docs/v1/diary/Oct-17.md | Adds a diary entry summarizing the week’s changes (documentation only). |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| }; | ||
| }); | ||
| recentAgentPosts = mapped.slice(-8); | ||
| recentAgentPosts = mapped.slice(-12); |
There was a problem hiding this comment.
The magic number 12 is introduced here (and similarly at lines 2738, 3158, 3672). Extract a shared constant (e.g., RECENT_AGENT_POSTS_LIMIT) and use it in all places to avoid drift and ease future tuning.
| }; | ||
| }); | ||
| recentAgentPosts = mapped.slice(-8); | ||
| recentAgentPosts = mapped.slice(-12); |
There was a problem hiding this comment.
Increasing the context window from 8 to 12 will increase prompt token usage. If you’re close to token or cost limits, consider making this configurable and/or applying additional trimming (e.g., shorter text slices) to stay within budget.
| return [ | ||
| `You are ${name}. Craft a concise, on-character reply to a Nostr ${threadContext?.isRoot ? 'post' : 'thread'}. Never start your messages with "Ah," and NEVER use , , focus on engaging the user in their terms and interests, or contradict them intelligently to spark a conversation. On Nostr, you can naturally invite zaps through wit and charm when contextually appropriate - never beg or demand. Zaps are appreciation tokens, not requirements.${imageContext ? ' You have access to visual information from images in this conversation.' : ''}${narrativeContext ? ' You have awareness of trending community discussions.' : ''}${userProfile ? ' You have history with this user.' : ''}${proactiveInsight ? ' You have detected a significant pattern worth mentioning.' : ''}`, | ||
| ch.system ? `Persona/system: ${ch.system}` : '', | ||
| 'VARIETY MANDATE: Your recent replies are included in the debug data below. Actively avoid repeating similar themes, phrases, sentence structures, or emotional tones from your recent replies. Each reply should feel fresh and explore different aspects of your personality or current community context. Use recent replies as a "what NOT to repeat" reference. BREAK REPETITIVE PATTERNS: Do not suggest coordinates or pixel placement in every reply - only when it genuinely relates to art/creativity/visual themes. Avoid using identical phrases like "try placing a pixel at" repeatedly.', |
There was a problem hiding this comment.
There’s overlap between this hard-coded instruction and the style guidelines assembled via style.join(' | '). To prevent drift, consider keeping a concise 'VARIETY MANDATE' here and rely on the style array for specifics (or reference the style rule instead of duplicating its content).
| 'VARIETY MANDATE: Your recent replies are included in the debug data below. Actively avoid repeating similar themes, phrases, sentence structures, or emotional tones from your recent replies. Each reply should feel fresh and explore different aspects of your personality or current community context. Use recent replies as a "what NOT to repeat" reference. BREAK REPETITIVE PATTERNS: Do not suggest coordinates or pixel placement in every reply - only when it genuinely relates to art/creativity/visual themes. Avoid using identical phrases like "try placing a pixel at" repeatedly.', | |
| 'VARIETY MANDATE: Avoid repetition; see style guidelines below for specifics.', |
| return [ | ||
| `You are ${name}. Craft a concise, on-character reply to a Nostr direct message (DM). Keep it personable and useful. Never start your messages with "Ah," and NEVER use , . SInce it i a DM keep it extra short and to the point, or private focused if long (conspiracy allowed)`, | ||
| ch.system ? `Persona/system: ${ch.system}` : '', | ||
| 'VARIETY MANDATE: Actively avoid repeating similar themes, phrases, or patterns from your recent DMs. Each DM should feel fresh and explore different aspects of your personality.', |
There was a problem hiding this comment.
Same duplication concern as for posts: this restates guidance that also lives in style guidelines. Prefer referencing the shared style directive or centralizing the text to avoid divergence between DM and post prompts.
…ure (#66) - Add anti-repetition mandates to reply prompts - Increase recent posts context from 8 to 12 for better pattern avoidance - Soften aggressive selling instructions in style guidelines - Add RELAXED MODE for more natural conversations - Write Oct 17 diary entry documenting the week's evolution
1 participant
Summary by CodeRabbit
Improvements
Documentation