Skip to content

Conversation

@bradflaugher
Copy link
Collaborator

@bradflaugher bradflaugher commented Nov 18, 2025

Problem

The current VICTORIA.md has a 'Prime Directive (Non-Negotiable)' focused on ratio metric calculations. While this is important, it's not really the prime directive for a helpful adtech assistant.

Solution

This PR reframes the Prime Directive to focus on Victoria's core purpose: being helpful, understanding user requests fully, and thinking step-by-step. The calculation rules become important guidelines rather than the prime directive itself.

Changes

VICTORIA.md

New Prime Directive:

Be helpful. Understand the user's request fully before acting. Think step-by-step, read the guidelines below, and apply best practices consistently. Your goal is to provide accurate, actionable insights that help users make better decisions.

Important Guidelines section includes:

  • Ratio Metrics Calculation (aggregate numerators/denominators first, use safe division)
  • Data Exploration tips
  • Integration password

CRUSH.md

  • Add Prime Directive to the 'Why This Matters' section
  • Update self-check questions to include Prime Directive understanding
  • Reframe calculation rules as 'ratio metrics calculation guideline'
  • Keep all the same technical emphasis

Impact

Victoria's primary focus is now on being helpful and understanding users, with technical best practices as important guidelines to follow. This better reflects her role as an adtech assistant while maintaining all the critical calculation rules.

Update the Prime Directive to focus on Victoria's core purpose: being helpful,
understanding user requests fully, and thinking step-by-step. The calculation
rules are now presented as important guidelines rather than the prime directive.

Changes to VICTORIA.md:
- New Prime Directive: 'Be helpful. Understand the user's request fully before
  acting. Think step-by-step, read the guidelines below, and apply best
  practices consistently.'
- Moved ratio metrics calculation to 'Important Guidelines' section
- Keeps all technical guidance about safe division and aggregation
- Maintains data exploration tips and integration password

Changes to CRUSH.md:
- Update references to align with new Prime Directive
- Add Prime Directive to the 'Why This Matters' section
- Update self-check questions to include Prime Directive understanding
- Reframe calculation rules as guidelines rather than directives

This presents Victoria as a helpful adtech assistant whose primary goal is
understanding and helping users, with technical best practices as important
guidelines to follow.
@bradflaugher bradflaugher force-pushed the refactor/improve-victoria-tone branch from ec1e5d7 to fb04fcc Compare November 18, 2025 13:55
@bradflaugher bradflaugher changed the title refactor: replace 'Prime Directive' with balanced tone refactor: reframe Prime Directive to focus on being helpful Nov 18, 2025
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Address reviewer feedback about conflicting Prime Directive definitions.
The Prime Directive is now consistently defined as 'be helpful' throughout
the codebase, with ratio calculation rules referred to as guidelines or
critical rules.

Changes:
- VICTORIA.md line 168: 'Prime Directive' -> 'ratio calculation guideline'
- VICTORIA.md line 740: 'Prime Directive' -> 'ratio calculation guideline'
- AGENTS.md: 'Prime Directive' -> 'Critical Calculation Rule'

This ensures agents have clear, consistent guidance:
- Prime Directive = be helpful, understand requests, think step-by-step
- Critical Calculation Rule/Guideline = aggregate before dividing ratios

Resolves the conflicting definitions that could cause incorrect calculations.
@bradflaugher bradflaugher merged commit ac36a05 into main Nov 18, 2025
1 of 2 checks passed
@bradflaugher bradflaugher deleted the refactor/improve-victoria-tone branch November 18, 2025 14:16
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