Skip to content

Commit 784be0a

Browse files
authored
docs: consistent casing, copy, grammar, spacing (#144)
1 parent 7a2188f commit 784be0a

File tree

10 files changed

+22
-25
lines changed

10 files changed

+22
-25
lines changed

CONTRIBUTING.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ your contribution is aligned with the project's goals.
2121
npm run test:integration
2222
npm start
2323
```
24-
All tests should pass, and the application should start successfully with confirmation messaging in the terminal.
24+
All tests should pass, and the application should start successfully with a confirmation message in the terminal.
2525

2626
##### Windows and repository symlinks
2727

@@ -48,7 +48,7 @@ Our process follows the standard GitHub fork and pull request workflow.
4848
##### Main repository branches
4949
- The `main` branch currently represents both development and stable releases
5050

51-
> In the future, a consideration for a `stable` branch may be made against an increase in contributions.
51+
> In the future, if there is an increase in contributions, we may consider implementing a `stable` branch.
5252
> - `main` would be the default branch for development and feature work rebased from `stable` after release.
5353
> - `stable` would be a branch used for stable releases/hashes, reference links, and only updated with release commits.
5454
@@ -93,8 +93,8 @@ Commit messages follow two basic guidelines:
9393
- **Description**: What the commit work encompasses.
9494
- **#PR_NUMBER**: The pull request number. Typically added automatically during merge/squash operations. Including it manually is optional. It can help with traceability during review.
9595

96-
> If your **pull request contains multiple commits**, they will be squashed into a single commit before merging and the messaging
97-
> altered to reflect current guidelines.
96+
> If your **pull request contains multiple commits**, they will be squashed into a single commit before merging, and the messaging
97+
> will be altered to reflect current guidelines.
9898
9999
#### Pull request test failures
100100
Before any review takes place, all tests should pass. You may be asked to update your pull request to resolve any failing tests

README.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
A Model Context Protocol (MCP) server that provides access to PatternFly rules and documentation, built with Node.js.
44

55
The PatternFly MCP server is a comprehensive library resource for PatternFly.
6-
It is intended to be extensible to meet the needs of different teams and projects, from simple to complex, from design to development.
6+
It is intended to be extensible to meet the needs of different teams and projects, from simple to complex, from design to development.
77
[Read more about our roadmap and how we've structured the server in our architecture docs](./docs/architecture.md).
88

99
## Requirements
@@ -68,7 +68,6 @@ Visualize and test the MCP interface:
6868
npx -y @modelcontextprotocol/inspector npx @patternfly/patternfly-mcp
6969
```
7070

71-
7271
#### Embed the server in your application
7372

7473
```typescript
@@ -94,7 +93,7 @@ main();
9493

9594
For comprehensive usage, development, and project state [read the docs](./docs/README.md).
9695

97-
- **Architecture**: Learn about our [hybrid documentation model and data sources](./docs/architecture.md#data-sources-and-integrations).
96+
- **Architecture**: Learn about our [hybrid documentation concept and data sources](./docs/architecture.md#data-sources-and-integrations).
9897
- **Usage**: Detailed guide on [built-in tools and resources](./docs/usage.md).
9998
- **Development**: Reference for [CLI options and tool plugins](./docs/development.md).
10099

docs/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,5 @@ Welcome to the PatternFly MCP Server documentation. This guide is organized by u
1212
- **[Examples](./examples/README.md)**: Standalone snippets for HTTP transport, embedding, and custom tools.
1313

1414
### 🏗️ Architecture & design
15-
- **[System Architecture](./architecture.md)**: Architecture and roadmap.
15+
- **[System Architecture](./architecture.md)**: Design and core concepts.
1616
- **[Roadmap](./architecture.md#roadmap)**: Future plans for the project.

docs/architecture.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ These helpers are a core part of our [Hybrid documentation](#hybrid-documentatio
4444

4545
### Tools, resources, and prompts as customizable plugins
4646

47-
Tools, resources, and prompts as customizable plugins are the result of predictable MCP SDK patterns. In the case of the PatternFly MCP server,
47+
Customizable plugins for tools, resources, and prompts follow predictable MCP SDK patterns. In the case of the PatternFly MCP server,
4848
this actively plays a role in the library architecture because it allows us to focus on providing stability.
4949

5050
Key goals aided by moving towards plugins:

docs/development.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ Complete guide to using the PatternFly MCP Server for development including CLI
3232

3333
#### Notes
3434
- **HTTP transport mode** - By default, the server uses `stdio`. Use the `--http` flag to enable HTTP transport.
35-
- **Logging** - The server uses a `diagnostics_channel`-based logger that keeps STDIO stdout pure by default.
35+
- **Logging** - The server uses a `diagnostics_channel`-based logger that keeps stdout pure by default.
3636
- **Programmatic API** - The server can also be used programmatically with options. See [Programmatic usage](#programmatic-usage) for more details.
3737
- **Tool plugins** - The server can load external tool plugins at startup. See [MCP tool plugins](#mcp-tool-plugins) for more details.
3838
- **Test Mode** - When `--mode test` is used, the server redirects resource requests to the URL provided by `--mode-test-url`, enabling E2E testing without local filesystem access.
@@ -205,7 +205,7 @@ process.on('SIGINT', async () => {
205205

206206
#### server.getStats()
207207

208-
Returns a `PfMcpStats` object containing server metrics, includes diagnostic channel IDs for listening to server activity.
208+
Returns a `PfMcpStats` object containing server metrics and diagnostic channel IDs for listening to server activity.
209209

210210
```typescript
211211
import { channel, unsubscribe, subscribe } from 'node:diagnostics_channel';
@@ -220,7 +220,6 @@ const logHandler = (event: PfMcpStatReport) => {
220220
};
221221

222222
const logSubscription = subscribe(logChannel, logHandler);
223-
224223
```
225224

226225
### Typing reference
@@ -254,8 +253,8 @@ const main = async () => {
254253
]
255254
});
256255

257-
// Optional: observe refined server logs inprocess.
258-
// We recommend getting in the habit of avoiding use of console.log and info, they pollute STDOUT.
256+
// Optional: observe refined server logs in-process.
257+
// We recommend getting in the habit of avoiding use of console.log and info since they pollute stdout.
259258
server.onLog((event) => {
260259
if (event.level !== 'debug') {
261260
console.warn(`[${event.level}] ${event.msg || ''}`);
@@ -276,7 +275,7 @@ See [examples/](examples/) for more programmatic usage examples.
276275

277276
## MCP tool plugins
278277

279-
You can extend the server's capabilities by loading **Tool Plugins** at startup. These plugins run out‑of‑process in an isolated **Tools Host** to ensure security and stability.
278+
You can extend the server's capabilities by loading **Tool Plugins** at startup. These plugins run out-of-process in an isolated **Tools Host** to ensure security and stability.
280279

281280
### Tool plugin runtime requirements
282281

docs/examples/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ PatternFly MCP code examples for tooling plugins and development use cases.
88
- **[toolPluginGitStatus.js](toolPluginGitStatus.js)** - A custom tool using Git
99
- **[toolPluginHelloWorld.js](toolPluginHelloWorld.js)** - A basic JS tool plugin example
1010

11-
1211
## Adding new example guidance
1312

1413
Examples should follow the basic guidelines:

docs/usage.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
A comprehensive guide to PatternFly MCP Server tools, resources, and configuration.
44

55
**User Guide:**
6-
- [Built-in Tools](#built-in-tools)
7-
- [Built-in Resources](#built-in-resources)
8-
- [MCP Client Configuration](#mcp-client-configuration)
9-
- [Custom MCP Tool Plugins](#custom-mcp-tool-plugins)
6+
- [Built-in tools](#built-in-tools)
7+
- [Built-in resources](#built-in-resources)
8+
- [MCP client configuration](#mcp-client-configuration)
9+
- [Custom MCP tool plugins](#custom-mcp-tool-plugins)
1010

1111
## Built-in tools
1212

guidelines/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,9 @@ Agents should use these phrases as signals to consult specific documentation and
4646

4747
### File Maintenance Principles
4848
- Update index files (e.g., `docs/examples/README.md` or `guidelines/README.md`) immediately when adding or removing content.
49-
- Reference and index guidelines. Don't duplicate content
50-
- Update references when adding new files
51-
- Keep descriptions concise and focused
49+
- Reference and index guidelines. Don't duplicate content.
50+
- Update references when adding new files.
51+
- Keep descriptions concise and focused.
5252

5353
### Adding New Guidelines
5454
1. Add entry to "Guidelines Index" section

guidelines/agent_behaviors.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Comprehensive guide to agent behaviors, workflows, and standards for the codebas
88

99
### Processing Priority
1010

11-
Critical - Process first when working with the repository.
11+
Critical - This document should be processed first when working with agent guidelines in the repository.
1212

1313
### Related Guidelines
1414

guidelines/agent_coding.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,5 +229,5 @@ Agents MUST validate all code outputs using the project's quality suite:
229229

230230
1. **Linting**: `npm run test:lint` (Ensures style consistency)
231231
2. **Type Checking**: `npm run test:types` (tsc validation)
232-
3. **Documentation**: `npm run test:spell-docs` (Cspell validation)
232+
3. **Documentation**: `npm run test:spell-docs` (CSpell validation)
233233
4. **Testing**: `npm run test` (Unit) and `npm run test:integration` (E2E)

0 commit comments

Comments
 (0)