Skip to content

Commit cc9ffe1

Browse files
Copilotjstirnaman
andauthored
Merge branch 'master' into copilot/rebase-on-master
Rebase PR #7038 on master: - Resolve notifications.yaml conflict (keep master's updated path and terminology) - Remove old admin/pachatree/ files (superseded by performance-preview/ in master) - Add ingest node note to performance-preview/configure.md - Add replication config link to performance-preview/monitor.md Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
2 parents 2ed446a + 1252d76 commit cc9ffe1

71 files changed

Lines changed: 2163 additions & 685 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.ci/vale/styles/InfluxDataDocs/Acronyms.yml

Lines changed: 100 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,106 @@ exceptions:
4949
- SCSS
5050
- SDK
5151
- SQL
52+
# SQL/InfluxQL keywords (3-5 uppercase chars trigger this rule)
53+
# These are standard uppercase query clauses, not acronyms.
54+
- ABS
55+
- ADD
56+
- ALL
57+
- AND
58+
- ANY
59+
- ARE
60+
- ASC
61+
- AVG
62+
- BIT
63+
- CASE
64+
- CAST
65+
- COS
66+
- DAY
67+
- DEC
68+
- DESC
69+
- DROP
70+
- ELSE
71+
- END
72+
- EXEC
73+
- EXP
74+
- FOR
75+
- FROM
76+
- FULL
77+
- GRANT
78+
- GROUP
79+
- INNER
80+
- INPUT
81+
- INT
82+
- INTO
83+
- JOIN
84+
- KEY
85+
- LAST
86+
- LEFT
87+
- LEVEL
88+
- LIKE
89+
- LOG
90+
- LOWER
91+
- MATCH
92+
- MAX
93+
- MIN
94+
- MONTH
95+
- NAME
96+
- NAMES
97+
- NOT
98+
- NOW
99+
- "NULL"
100+
- "ON"
101+
- ONLY
102+
- OPEN
103+
- ORDER
104+
- OUTER
105+
- PAD
106+
- POW
107+
- PRIOR
108+
- READ
109+
- REAL
110+
- RIGHT
111+
- ROWS
112+
- SET
113+
- SIN
114+
- SIZE
115+
- SOME
116+
- SPACE
117+
- SUM
118+
- TABLE
119+
- TAN
120+
- THEN
121+
- TIME
122+
- TOP
123+
- TRIM
124+
- "TRUE"
125+
- UNION
126+
- UPPER
127+
- USAGE
128+
- USING
129+
- VALUE
130+
- VIEW
131+
- WHEN
132+
- WHERE
133+
- WITH
134+
- WORK
135+
- WRITE
136+
- YEAR
137+
- ZONE
138+
# InfluxQL-specific
139+
- COUNT
140+
- FILL
141+
- FIRST
142+
- FLOAT
143+
- MEAN
144+
- BEGIN
145+
- ALTER
146+
- CHECK
147+
- CLOSE
148+
- CROSS
149+
- FETCH
150+
- LIMIT
151+
- SHOW
52152
- SSH
53153
- SSL
54154
- SVG

.claude/commands/finish.md

Lines changed: 14 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Complete development work by cleaning up ephemeral documents and preparing for m
66

77
This skill handles the end of a development workflow:
88
1. Reads the full contents of PLAN.md
9-
2. Adds the complete plan details to the PR description (or as a PR comment if the description is already long)
9+
2. Posts the complete plan details as a PR comment
1010
3. Removes ephemeral planning documents
1111
4. Creates a cleanup commit
1212
5. Optionally merges the PR
@@ -23,7 +23,7 @@ This skill handles the end of a development workflow:
2323

2424
- `--merge`: After cleanup, merge the PR using `gh pr merge --squash`
2525
- `--dry-run`: Preview actions without executing them
26-
- `--no-pr-update`: Skip updating PR description
26+
- `--no-comment`: Skip posting plan details as PR comment
2727

2828
## Steps
2929

@@ -59,42 +59,12 @@ Read the entire PLAN.md file and preserve all sections:
5959
PLAN_CONTENTS="$(cat PLAN.md)"
6060
```
6161

62-
### 3. Add Plan Details to PR
62+
### 3. Post Plan Details as PR Comment
6363

64-
Add the **complete** PLAN.md contents to the PR. Do not summarize or abbreviate
65-
the plan -- include all details so they are preserved after the file is deleted.
66-
67-
**Strategy:**
68-
- If the PR description is short (under ~2000 characters), append the full plan
69-
to the PR description.
70-
- If the PR description is already long, post the full plan as a PR comment
71-
instead, to keep the description readable.
72-
73-
**Appending to PR description:**
74-
75-
```bash
76-
# Get current PR body
77-
CURRENT_BODY="$(gh pr view --json body -q '.body')"
78-
79-
# Update PR with full plan details appended
80-
gh pr edit --body "$(cat <<EOF
81-
${CURRENT_BODY}
82-
83-
---
84-
85-
## Development Plan
86-
87-
<details>
88-
<summary>Full plan details (from PLAN.md)</summary>
89-
90-
${PLAN_CONTENTS}
91-
92-
</details>
93-
EOF
94-
)"
95-
```
96-
97-
**Posting as PR comment (if description is already long):**
64+
Post the **complete** PLAN.md contents as a PR comment. Do not summarize or
65+
abbreviate the plan -- include all details so they are preserved after the file
66+
is deleted. Always use a comment (not the PR description) to keep the
67+
description clean and editable.
9868

9969
```bash
10070
gh pr comment --body "$(cat <<EOF
@@ -182,7 +152,7 @@ Agent:
182152
183153
### Actions:
184154
1. Read full PLAN.md contents (42 lines, all sections preserved)
185-
2. Updated PR #6789 description with complete plan details
155+
2. Posted full plan details as comment on PR #6789
186156
3. Removed PLAN.md
187157
4. Created commit: "chore: remove planning docs"
188158
5. Pushed to origin
@@ -205,29 +175,7 @@ Agent:
205175
Done! PR merged successfully.
206176
```
207177

208-
### Example 3: Plan added as comment (long PR description)
209-
```
210-
User: /finish
211-
212-
Agent:
213-
## Finishing development work
214-
215-
### Prerequisites
216-
- Branch: docs-v2-issue6763
217-
- PLAN.md: Found
218-
- PR description: 3200 characters (long)
219-
220-
### Actions:
221-
1. Read full PLAN.md contents
222-
2. PR description already long -- posted full plan as PR comment on #6789
223-
3. Removed PLAN.md
224-
4. Created commit: "chore: remove planning docs"
225-
5. Pushed to origin
226-
227-
Ready for merge!
228-
```
229-
230-
### Example 4: Incomplete tasks warning
178+
### Example 3: Incomplete tasks warning
231179
```
232180
User: /finish
233181
@@ -246,7 +194,7 @@ Agent: Proceeding with cleanup...
246194
[continues with normal flow]
247195
```
248196

249-
### Example 5: Dry run
197+
### Example 4: Dry run
250198
```
251199
User: /finish --dry-run
252200
@@ -255,7 +203,7 @@ Agent:
255203
256204
Would perform the following actions:
257205
1. Read full PLAN.md contents
258-
2. Add complete plan details to PR #6789 description (or comment)
206+
2. Post full plan details as comment on PR #6789
259207
3. Run: git rm PLAN.md
260208
4. Run: git commit -m "chore: remove planning docs"
261209
5. Run: git push
@@ -266,10 +214,9 @@ No changes made. Run `/finish` to execute.
266214
## Notes
267215

268216
- Always reads full PLAN.md contents before deleting it
269-
- Preserves complete plan details (not just a summary) in the PR for future reference
270-
- If the PR description is short, appends plan details in a collapsible `<details>` block
271-
- If the PR description is already long, posts the full plan as a PR comment instead
217+
- Preserves complete plan details (not just a summary) as a PR comment for future reference
218+
- Always uses a PR comment, keeping the PR description clean and editable
272219
- Squash merge is recommended to keep main branch clean
273220
- The deleted PLAN.md remains in branch history (recoverable if needed)
274221
- Works with GitHub Actions cleanup as a fallback safety net
275-
- Use `--no-pr-update` if you want to write the PR description manually
222+
- Use `--no-comment` to skip posting plan details to the PR

.claude/launch.json

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
{
2+
"version": "0.0.1",
3+
"configurations": [
4+
{
5+
"name": "docs-dev",
6+
"runtimeExecutable": "npx",
7+
"runtimeArgs": ["hugo", "server"],
8+
"port": 1313
9+
},
10+
{
11+
"name": "docs-test",
12+
"runtimeExecutable": "npx",
13+
"runtimeArgs": ["hugo", "server", "--environment", "testing", "--port", "1315", "--noHTTPCache"],
14+
"port": 1315
15+
}
16+
]
17+
}

.claude/settings.json

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -107,25 +107,13 @@
107107
]
108108
}
109109
],
110-
"WorktreeCreate": [
111-
{
112-
"hooks": [
113-
{
114-
"type": "command",
115-
"command": "CYPRESS_INSTALL_BINARY=0 yarn install --frozen-lockfile 2>&1 | tail -3",
116-
"timeout": 120,
117-
"statusMessage": "Installing dependencies in new worktree"
118-
}
119-
]
120-
}
121-
],
122110
"SessionStart": [
123111
{
124112
"hooks": [
125113
{
126114
"type": "command",
127-
"command": "[ ! -d node_modules ] && CYPRESS_INSTALL_BINARY=0 yarn install --frozen-lockfile 2>&1 | tail -3 || true",
128-
"timeout": 120,
115+
"command": "[ ! -d node_modules ] && CYPRESS_INSTALL_BINARY=0 PUPPETEER_SKIP_DOWNLOAD=1 PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 yarn install --frozen-lockfile 2>&1 | tail -3 || true",
116+
"timeout": 300,
129117
"async": true,
130118
"statusMessage": "Checking dependencies"
131119
}

0 commit comments

Comments
 (0)