ep_chat_log_join_leave 1.0.49 → 1.0.51
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -34,7 +34,6 @@ jobs:
|
|
|
34
34
|
- uses: pnpm/action-setup@v6
|
|
35
35
|
name: Install pnpm
|
|
36
36
|
with:
|
|
37
|
-
version: 10
|
|
38
37
|
run_install: false
|
|
39
38
|
- name: Get pnpm store directory
|
|
40
39
|
shell: bash
|
|
@@ -53,10 +52,21 @@ jobs:
|
|
|
53
52
|
fetch-depth: 0
|
|
54
53
|
-
|
|
55
54
|
name: Bump version (patch)
|
|
55
|
+
id: bump
|
|
56
56
|
run: |
|
|
57
57
|
LATEST_TAG=$(git describe --tags --abbrev=0) || exit 1
|
|
58
58
|
NEW_COMMITS=$(git rev-list --count "${LATEST_TAG}"..) || exit 1
|
|
59
|
-
|
|
59
|
+
# No new commits since the last tag → nothing to publish.
|
|
60
|
+
# Setting `bumped=false` lets the publish step below skip
|
|
61
|
+
# itself instead of trying to republish the existing version
|
|
62
|
+
# (which fails with "cannot publish over previously published
|
|
63
|
+
# versions"). Only manual `gh workflow run` invocations on a
|
|
64
|
+
# tag-current branch hit this path; on:push always sees ≥1
|
|
65
|
+
# new commit because the push event is what triggered us.
|
|
66
|
+
if [ "${NEW_COMMITS}" -le 0 ]; then
|
|
67
|
+
echo "bumped=false" >> "${GITHUB_OUTPUT}"
|
|
68
|
+
exit 0
|
|
69
|
+
fi
|
|
60
70
|
git config user.name 'github-actions[bot]'
|
|
61
71
|
git config user.email '41898282+github-actions[bot]@users.noreply.github.com'
|
|
62
72
|
pnpm i --frozen-lockfile
|
|
@@ -84,9 +94,11 @@ jobs:
|
|
|
84
94
|
# rejects the tag push too, and the next workflow tick can retry
|
|
85
95
|
# cleanly against the up-to-date refs.
|
|
86
96
|
git push --atomic origin "${GITHUB_REF_NAME}" "${NEW_TAG}"
|
|
97
|
+
echo "bumped=true" >> "${GITHUB_OUTPUT}"
|
|
87
98
|
# This is required if the package has a prepare script that uses something
|
|
88
99
|
# in dependencies or devDependencies.
|
|
89
100
|
-
|
|
101
|
+
if: steps.bump.outputs.bumped == 'true'
|
|
90
102
|
run: pnpm i
|
|
91
103
|
# `npm publish` must come after `git push` otherwise there is a race
|
|
92
104
|
# condition: If two PRs are merged back-to-back then master/main will be
|
|
@@ -104,4 +116,5 @@ jobs:
|
|
|
104
116
|
# whichever `npm` is on PATH; calling `npm` directly avoids any shim
|
|
105
117
|
# ambiguity.
|
|
106
118
|
- name: Publish to npm via OIDC
|
|
119
|
+
if: steps.bump.outputs.bumped == 'true'
|
|
107
120
|
run: npm publish --provenance --access public
|
package/CONTRIBUTING.md
CHANGED
|
@@ -1,135 +1,74 @@
|
|
|
1
|
-
#
|
|
2
|
-
(Please talk to people on the mailing list before you change this page, see our section on [how to get in touch](https://github.com/ether/etherpad-lite#get-in-touch))
|
|
3
|
-
|
|
4
|
-
## Pull requests
|
|
5
|
-
|
|
6
|
-
* the commit series in the PR should be _linear_ (it **should not contain merge commits**). This is necessary because we want to be able to [bisect](https://en.wikipedia.org/wiki/Bisection_(software_engineering)) bugs easily. Rewrite history/perform a rebase if necessary
|
|
7
|
-
* PRs should be issued against the **develop** branch: we never pull directly into **master**
|
|
8
|
-
* PRs **should not have conflicts** with develop. If there are, please resolve them rebasing and force-pushing
|
|
9
|
-
* when preparing your PR, please make sure that you have included the relevant **changes to the documentation** (preferably with usage examples)
|
|
10
|
-
* contain meaningful and detailed **commit messages** in the form:
|
|
11
|
-
```
|
|
12
|
-
submodule: description
|
|
13
|
-
|
|
14
|
-
longer description of the change you have made, eventually mentioning the
|
|
15
|
-
number of the issue that is being fixed, in the form: Fixes #someIssueNumber
|
|
16
|
-
```
|
|
17
|
-
* if the PR is a **bug fix**:
|
|
18
|
-
* the first commit in the series must be a test that shows the failure
|
|
19
|
-
* subsequent commits will fix the bug and make the test pass
|
|
20
|
-
* the final commit message should include the text `Fixes: #xxx` to link it to its bug report
|
|
21
|
-
* think about stability: code has to be backwards compatible as much as possible. Always **assume your code will be run with an older version of the DB/config file**
|
|
22
|
-
* if you want to remove a feature, **deprecate it instead**:
|
|
23
|
-
* write an issue with your deprecation plan
|
|
24
|
-
* output a `WARN` in the log informing that the feature is going to be removed
|
|
25
|
-
* remove the feature in the next version
|
|
26
|
-
* if you want to add a new feature, put it under a **feature flag**:
|
|
27
|
-
* once the new feature has reached a minimal level of stability, do a PR for it, so it can be integrated early
|
|
28
|
-
* expose a mechanism for enabling/disabling the feature
|
|
29
|
-
* the new feature should be **disabled** by default. With the feature disabled, the code path should be exactly the same as before your contribution. This is a __necessary condition__ for early integration
|
|
30
|
-
* think of the PR not as something that __you wrote__, but as something that __someone else is going to read__. The commit series in the PR should tell a novice developer the story of your thoughts when developing it
|
|
31
|
-
|
|
32
|
-
## How to write a bug report
|
|
33
|
-
|
|
34
|
-
* Please be polite, we all are humans and problems can occur.
|
|
35
|
-
* Please add as much information as possible, for example
|
|
36
|
-
* client os(s) and version(s)
|
|
37
|
-
* browser(s) and version(s), is the problem reproducible on different clients
|
|
38
|
-
* special environments like firewalls or antivirus
|
|
39
|
-
* host os and version
|
|
40
|
-
* npm and nodejs version
|
|
41
|
-
* Logfiles if available
|
|
42
|
-
* steps to reproduce
|
|
43
|
-
* what you expected to happen
|
|
44
|
-
* what actually happened
|
|
45
|
-
* Please format logfiles and code examples with markdown see github Markdown help below the issue textarea for more information.
|
|
46
|
-
|
|
47
|
-
If you send logfiles, please set the loglevel switch DEBUG in your settings.json file:
|
|
1
|
+
# Contributing to ep_chat_log_join_leave
|
|
48
2
|
|
|
49
|
-
|
|
50
|
-
/* The log level we are using, can be: DEBUG, INFO, WARN, ERROR */
|
|
51
|
-
"loglevel": "DEBUG",
|
|
52
|
-
```
|
|
3
|
+
Thanks for helping improve `ep_chat_log_join_leave` — a plugin for [Etherpad](https://etherpad.org/). LLM/Agent contributions are explicitly welcome, provided they follow the rules below.
|
|
53
4
|
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
## General goals of Etherpad
|
|
57
|
-
To make sure everybody is going in the same direction:
|
|
58
|
-
* easy to install for admins and easy to use for people
|
|
59
|
-
* easy to integrate into other apps, but also usable as standalone
|
|
60
|
-
* lightweight and scalable
|
|
61
|
-
* extensible, as much functionality should be extendable with plugins so changes don't have to be done in core.
|
|
62
|
-
Also, keep it maintainable. We don't wanna end up as the monster Etherpad was!
|
|
63
|
-
|
|
64
|
-
## How to work with git?
|
|
65
|
-
* Don't work in your master branch.
|
|
66
|
-
* Make a new branch for every feature you're working on. (This ensures that you can work you can do lots of small, independent pull requests instead of one big one with complete different features)
|
|
67
|
-
* Don't use the online edit function of github (this only creates ugly and not working commits!)
|
|
68
|
-
* Try to make clean commits that are easy readable (including descriptive commit messages!)
|
|
69
|
-
* Test before you push. Sounds easy, it isn't!
|
|
70
|
-
* Don't check in stuff that gets generated during build or runtime
|
|
71
|
-
* Make small pull requests that are easy to review but make sure they do add value by themselves / individually
|
|
5
|
+
For shared rules that apply to all Etherpad code (linear commits, deprecation policy, feature flags, stability guarantees), please read [ether/etherpad's CONTRIBUTING.md](https://github.com/ether/etherpad/blob/develop/CONTRIBUTING.md). This document only covers what's specific to plugin work.
|
|
72
6
|
|
|
73
|
-
##
|
|
74
|
-
|
|
75
|
-
*
|
|
76
|
-
*
|
|
77
|
-
*
|
|
78
|
-
*
|
|
79
|
-
*
|
|
80
|
-
|
|
81
|
-
|
|
7
|
+
## Plugin-specific rules
|
|
8
|
+
|
|
9
|
+
* **Target branch:** PRs go to `main` on this repo. Plugin repos do not use git-flow's `develop` branch.
|
|
10
|
+
* **Linear commits, no merge commits.** Rebase if needed.
|
|
11
|
+
* **Every bug fix must include a regression test in the same commit.**
|
|
12
|
+
* **Internationalization (i18n):** every user-facing string lives in `locales/<lang>.json` and is referenced via `data-l10n-id` in templates or `html10n.get(key)` in code. No hardcoded English in markup.
|
|
13
|
+
* **Accessibility (a11y):** no nested interactive elements (no `<button>` inside `<a>`); every interactive control has an accessible name; keyboard focus order matches visual order. Note that on icon-only controls you should rely on Etherpad's html10n to populate `aria-label` from the `data-l10n-id` translation — adding a hardcoded English `aria-label` blocks that and leaves the control untranslated.
|
|
14
|
+
|
|
15
|
+
## Local development
|
|
16
|
+
|
|
17
|
+
`ep_chat_log_join_leave` is a plugin, so you develop it inside a checkout of [ether/etherpad](https://github.com/ether/etherpad).
|
|
18
|
+
|
|
19
|
+
```bash
|
|
20
|
+
# Once: clone etherpad alongside this repo
|
|
21
|
+
git clone https://github.com/ether/etherpad.git
|
|
22
|
+
cd etherpad
|
|
23
|
+
pnpm install
|
|
82
24
|
|
|
83
|
-
|
|
84
|
-
|
|
25
|
+
# Install ep_chat_log_join_leave from your local clone
|
|
26
|
+
pnpm run plugins i --path ../ep_chat_log_join_leave
|
|
85
27
|
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
28
|
+
# Start the dev server (port 9001)
|
|
29
|
+
pnpm --filter ep_etherpad-lite run dev
|
|
30
|
+
```
|
|
31
|
+
|
|
32
|
+
Edits in `../ep_chat_log_join_leave/` are picked up after a server restart.
|
|
89
33
|
|
|
90
|
-
|
|
91
|
-
* everything that is READY to go into master at some point in time
|
|
92
|
-
* This stuff is tested and ready to go out
|
|
34
|
+
## Tests
|
|
93
35
|
|
|
94
|
-
|
|
95
|
-
* stuff that should go into master very soon
|
|
96
|
-
* only bugfixes go into these (see http://nvie.com/posts/a-successful-git-branching-model/ for why)
|
|
97
|
-
* we should not be blocking new features to develop, just because we feel that we should be releasing it to master soon. This is the situation that release branches solve/handle.
|
|
36
|
+
Tests live inside this repo and are executed by Etherpad's test harness:
|
|
98
37
|
|
|
99
|
-
|
|
100
|
-
*
|
|
38
|
+
* Backend (Mocha): `static/tests/backend/specs/`
|
|
39
|
+
* Playwright (frontend E2E): `static/tests/frontend-new/specs/`
|
|
101
40
|
|
|
102
|
-
|
|
103
|
-
* these are the branches where you develop your features in
|
|
104
|
-
* If it's ready to go out, it will be merged into develop
|
|
41
|
+
Run them from the etherpad checkout:
|
|
105
42
|
|
|
106
|
-
|
|
43
|
+
```bash
|
|
44
|
+
# Backend (no separate server needed, harness boots one)
|
|
45
|
+
pnpm --filter ep_etherpad-lite run test
|
|
46
|
+
|
|
47
|
+
# Playwright (requires `pnpm run dev` in another terminal)
|
|
48
|
+
pnpm --filter ep_etherpad-lite run test-ui
|
|
49
|
+
```
|
|
107
50
|
|
|
108
|
-
|
|
109
|
-
The docs are in the `doc/` folder in the git repository, so people can easily find the suitable docs for the current git revision.
|
|
51
|
+
Lint locally before pushing:
|
|
110
52
|
|
|
111
|
-
|
|
53
|
+
```bash
|
|
54
|
+
pnpm run lint
|
|
55
|
+
```
|
|
56
|
+
|
|
57
|
+
## Coding style
|
|
112
58
|
|
|
113
|
-
|
|
59
|
+
* JS/CSS/HTML: 2 spaces, no tabs.
|
|
60
|
+
* `pnpm run lint` must pass (configured via `eslint-config-etherpad`).
|
|
61
|
+
* Avoid breaking changes to public hooks, the line-attribute storage shape, or HTML export output. The backend export spec enforces a stable shape.
|
|
114
62
|
|
|
115
|
-
##
|
|
63
|
+
## Bug reports
|
|
116
64
|
|
|
117
|
-
|
|
118
|
-
Run them by pointing your browser to `<yourdomainhere>/tests/frontend`.
|
|
65
|
+
Please include:
|
|
119
66
|
|
|
120
|
-
|
|
67
|
+
* Etherpad version, ep_chat_log_join_leave version, browser + OS
|
|
68
|
+
* Steps to reproduce
|
|
69
|
+
* What you expected, what actually happened
|
|
70
|
+
* Server log excerpt (set `"loglevel": "DEBUG"` in `settings.json` for verbose output)
|
|
121
71
|
|
|
122
|
-
##
|
|
123
|
-
Etherpad is much more than software. So if you aren't a developer then worry not, there is still a LOT you can do! A big part of what we do is community engagement. You can help in the following ways
|
|
124
|
-
* Triage bugs (applying labels) and confirming their existence
|
|
125
|
-
* Testing fixes (simply applying them and seeing if it fixes your issue or not) - Some git experience required
|
|
126
|
-
* Notifying large site admins of new releases
|
|
127
|
-
* Writing Changelogs for releases
|
|
128
|
-
* Creating Windows packages
|
|
129
|
-
* Creating releases
|
|
130
|
-
* Bumping dependencies periodically and checking they don't break anything
|
|
131
|
-
* Write proposals for grants
|
|
132
|
-
* Co-Author and Publish CVEs
|
|
133
|
-
* Work with SFC to maintain legal side of project
|
|
134
|
-
* Maintain TODO page - https://github.com/ether/etherpad-lite/wiki/TODO#IMPORTANT_TODOS
|
|
72
|
+
## AI agents
|
|
135
73
|
|
|
74
|
+
If you are an LLM/agent contributor, also read [`AGENTS.md`](AGENTS.md) — it documents the file layout, helper usage, and standing rules for autonomous edits.
|