Skip to content

feat(cli): title the AGENTS.md block Passion Factory Code Style - #67

Merged
amondnet merged 2 commits into
mainfrom
amondnet/agents-heading-passion-factory
Oct 23, 2026
Merged

amondnet merged 2 commits into
mainfrom
amondnet/agents-heading-passion-factory

Conversation

@amondnet

@amondnet amondnet commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The managed AGENTS.md block heading changes from # PleaseAI Code Style to # Passion Factory Code Style, so the block carries the organization's name.

  • packages/cli/src/appliers.ts emits the new heading.
  • The example block in apps/docs/src/content/docs/{en,ko}/cli.mdx matches it.
  • One new test in packages/cli/test/appliers.test.ts asserts the heading.
  • The docs site uses the same name: the site title and preview image alt text in apps/docs/astro.config.ts, and the page title and Hero title in apps/docs/src/content/docs/{en,ko}/index.mdx.

The CLI finds the block by its <!-- pleaseai-code-style:start/end --> markers, not by the heading. update therefore replaces the old heading in existing projects with no migration step.

Out of scope, unchanged: descriptions and body text that say "PleaseAI projects", and ADR-0002.

Related issue

Refs #65 (the issue has remaining work: language detection and a no-package.json mode, so this PR does not close it).

Checklist

  • PR title follows Conventional Commits
  • Packages build (bun run build) – not run for the packages; the docs site builds (apps/docs: 23 pages)
  • Lint/format pass – eslint clean on the TS files; lint:docs clean (22 files)
  • Documentation updated if behavior changed
  • No breaking change

Verification

  • bun test in packages/cli: 253 pass, 1 skip, 0 fail.
  • tsc --noEmit: clean.

Summary by cubic

Titles the managed AGENTS.md block heading "Passion Factory Code Style" so the block carries the organization's name. The CLI locates the block by its marker comments rather than the heading, so update replaces the old heading in existing projects with no migration step.

The English and Korean docs examples match the new heading, and a test asserts it.

Refs #65 — the issue still has open work (language detection and a no-package.json mode), so this PR does not close it.

Written for commit fe741f0. Summary will update on new commits.

Turn on auto-fix

Rename the managed AGENTS.md block heading from "PleaseAI Code Style"
to "Passion Factory Code Style". The block is located by its marker
comments, not by the heading, so `update` replaces the old heading in
existing projects.

Refs #65
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Deploying code-style with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6adede5
Status: ✅  Deploy successful!
Preview URL: https://de0149e3.code-style.pages.dev
Branch Preview URL: https://amondnet-agents-heading-pass.code-style.pages.dev

View logs

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Match the site title, Hero title, and preview image alt text to the
renamed AGENTS.md block heading.

Refs #65
@sonarqubecloud

Copy link
Copy Markdown

@amondnet
amondnet marked this pull request as ready for review October 23, 2026 08:18

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 7 files

Architecture diagram
sequenceDiagram
    participant DEV as Developer
    participant CLI as @pleaseai/code-style CLI
    participant FS as File System (AGENTS.md)
    participant DOCS as Docs Site (Starlight)
    
    Note over DEV,FS: NEW: Managed AGENTS.md Block Heading
    
    DEV->>CLI: Run applyAgentsMd (init/update)
    CLI->>CLI: Generate block with NEW heading<br/>"# Passion Factory Code Style"
    CLI->>FS: Write marker-delimited block<br/><!-- pleaseai-code-style:start/end -->
    FS-->>CLI: Confirmation
    
    alt Existing AGENTS.md (update)
        CLI->>FS: Find block by marker comments (not heading)
        CLI->>FS: CHANGED: Replace old "PleaseAI Code Style" heading<br/>with new "Passion Factory Code Style"
    end
    
    CLI-->>DEV: Apply result
    
    Note over DOCS: Docs Site Content Updates
    
    DEV->>DOCS: Access documentation pages
    DOCS->>DOCS: CHANGED: Site title and social image alt<br/>use "Passion Factory Code Style"
    DOCS->>DOCS: CHANGED: Page titles and Hero titles<br/>(en/ko index.mdx, cli.mdx) updated
    DOCS-->>DEV: Render docs with consistent naming
    
    Note over CLI,FS: Test Verification
    
    DEV->>CLI: Run test suite
    CLI->>FS: Apply AGENTS.md block
    CLI->>CLI: NEW: Assert heading matches<br/>"# Passion Factory Code Style"
    CLI-->>DEV: All tests pass (253 pass, 1 skip)
Loading

Turn on auto-fix | Re-trigger cubic

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the branding of the CLI documentation and internal code templates by renaming 'PleaseAI Code Style' to 'Passion Factory Code Style'. The changes include updates to documentation files, the CLI applier logic, and the addition of a corresponding test case to verify the new title. I have no feedback to provide as the changes are straightforward and consistent across the codebase.

@greptile-apps

greptile-apps Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] This PR appears safe to merge.

Summary

Renames the managed AGENTS.md heading and the English and Korean docs titles to “Passion Factory Code Style.”

  • CLI-generated AGENTS.md blocks now use the Passion Factory name.
  • The documentation site and its pages use the Passion Factory name.

Acknowledged scope from amondnet: descriptions and body text mentioning “PleaseAI projects,” plus ADR-0002, intentionally remain unchanged.

Reviews (1) · Last reviewed commit: "docs: title the docs site Passion Factor..." · Reviewed by Greptile

@amondnet
amondnet merged commit ef4e526 into main Oct 23, 2026
13 checks passed
@amondnet
amondnet deleted the amondnet/agents-heading-passion-factory branch October 23, 2026 08:42
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.

1 participant