From e088fcf40b299d58cf0f2d0cca2c6b4100e4703a Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Wed, 1 Mar 2023 14:27:44 -0300
Subject: [PATCH 01/12] Test to verify if a command is documented

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/index.md                           |  1 +
 docs/occ.md                             |  2 +
 tests/php/Command/DocumentationTest.php | 52 +++++++++++++++++++++++++
 3 files changed, 55 insertions(+)
 create mode 100644 docs/occ.md
 create mode 100644 tests/php/Command/DocumentationTest.php

diff --git a/docs/index.md b/docs/index.md
index 4312cfb3f75..bd433aa5972 100644
--- a/docs/index.md
+++ b/docs/index.md
@@ -32,3 +32,4 @@
 ## Other
 
 * [Chat commands](commands.md)
+* [Occ commands](occ.md)
diff --git a/docs/occ.md b/docs/occ.md
new file mode 100644
index 00000000000..d9ab4d74b52
--- /dev/null
+++ b/docs/occ.md
@@ -0,0 +1,2 @@
+# Occ commands
+
diff --git a/tests/php/Command/DocumentationTest.php b/tests/php/Command/DocumentationTest.php
new file mode 100644
index 00000000000..462d2375762
--- /dev/null
+++ b/tests/php/Command/DocumentationTest.php
@@ -0,0 +1,52 @@
+<?php
+/**
+ * @copyright Copyright (c) 2023 Vitor Mattos <vitor@php.rio>
+ *
+ * @author Vitor Mattos <vitor@php.rio>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+namespace OCA\Talk\Tests\php\Command;
+
+use OCP\App\IAppManager;
+use Test\TestCase;
+
+final class DocumentationTest extends TestCase {
+	public function testCommandIsDocumented(): void {
+		/** @var IAppManager */
+		$appManager = \OC::$server->get(IAppManager::class);
+		$info = $appManager->getAppInfo('spreed');
+		foreach ($info['commands'] as $commandNamespace) {
+			$path = str_replace('OCA\Talk', '', $commandNamespace);
+			$path = str_replace('\\', '/', $path);
+			$path = __DIR__ . '/../../../lib' . $path . '.php';
+			$this->assertFileExists($path, 'The class of the follow namespase is not implemented: ' . $commandNamespace);
+			$code = file_get_contents($path);
+			preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
+			if (!array_key_exists('command', $matches)) {
+				preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
+			}
+			$this->assertArrayHasKey('command', $matches, 'A command need to have a name.');
+			$this->commandIsDocummented($matches['command']);
+		}
+	}
+
+	public function commandIsDocummented(string $command): void {
+		$docs = file_get_contents(__DIR__ . '/../../../docs/occ.md');
+		self::assertStringContainsString($command, $docs, 'Asserting that command ' . $command . ' is documented');
+	}
+}

From 0b5268ebdc46deefc15d8cc043fdd5244795d030 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Wed, 1 Mar 2023 21:40:01 -0300
Subject: [PATCH 02/12] Document commands

* created developer command to document commands

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 appinfo/info.xml                        |    1 +
 docs/occ.md                             | 1001 ++++++++++++++++++++++-
 lib/Command/Developer/UpdateDocs.php    |  168 ++++
 tests/php/Command/DocumentationTest.php |    6 +-
 tests/psalm-baseline.xml                |  111 ++-
 5 files changed, 1229 insertions(+), 58 deletions(-)
 create mode 100644 lib/Command/Developer/UpdateDocs.php

diff --git a/appinfo/info.xml b/appinfo/info.xml
index 7999b080f4d..ab7aa29279e 100644
--- a/appinfo/info.xml
+++ b/appinfo/info.xml
@@ -85,6 +85,7 @@ And in the works for the [coming versions](https://github.com/nextcloud/spreed/m
 		<command>OCA\Talk\Command\Command\Delete</command>
 		<command>OCA\Talk\Command\Command\ListCommand</command>
 		<command>OCA\Talk\Command\Command\Update</command>
+		<command>OCA\Talk\Command\Developer\UpdateDocs</command>
 		<command>OCA\Talk\Command\Monitor\Calls</command>
 		<command>OCA\Talk\Command\Monitor\HasActiveCalls</command>
 		<command>OCA\Talk\Command\Monitor\Room</command>
diff --git a/docs/occ.md b/docs/occ.md
index d9ab4d74b52..07379b86540 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -1,2 +1,1001 @@
-# Occ commands
+## talk:command:add
+
+Add a new command
+
+### Usage
+
+* `talk:command:add <cmd> <name> <script> <response> <enabled>`
+
+### Arguments
+
+#### `cmd`
+
+The command as used in the chat "/help" => "help"
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `name`
+
+Name of the user posting the response
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `script`
+
+Script to execute (Must be using absolute paths only)
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `response`
+
+Who should see the response: 0 - No one, 1 - User, 2 - All
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `enabled`
+
+Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:command:add-samples
+
+Adds some sample commands: /wiki, …
+
+### Usage
+
+* `talk:command:add-samples`
+
+
+## talk:command:delete
+
+Remove an existing command
+
+### Usage
+
+* `talk:command:delete <command-id>`
+
+### Arguments
+
+#### `command-id`
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:command:list
+
+List all available commands
+
+### Usage
+
+* `talk:command:list [--output [OUTPUT]] [--] [<app>]`
+
+### Arguments
+
+#### `app`
+
+Only list the commands of a specific app, "custom" to list all custom commands
+
+* Is required: no
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:command:update
+
+Add a new command
+
+### Usage
+
+* `talk:command:update <command-id> <cmd> <name> <script> <response> <enabled>`
+
+### Arguments
+
+#### `command-id`
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `cmd`
+
+The command as used in the chat "/help" => "help"
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `name`
+
+Name of the user posting the response
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `script`
+
+Script to execute (Must be using absolute paths only)
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `response`
+
+Who should see the response: 0 - No one, 1 - User, 2 - All
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `enabled`
+
+Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:developer:update-docs
+
+Update documentation of commands
+
+### Usage
+
+* `talk:developer:update-docs [--output [OUTPUT]]`
+
+### Arguments
+
+#### `command`
+
+The command to execute
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+#### `--help|-h`
+
+Display help for the given command. When no command is given display help for the list command
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--quiet|-q`
+
+Do not output any message
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--verbose|-v|-vv|-vvv`
+
+Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--version|-V`
+
+Display this application version
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--ansi|--no-ansi`
+
+Force (or disable --no-ansi) ANSI output
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: yes
+* Default: `NULL`
+
+#### `--no-interaction|-n`
+
+Do not ask any interactive question
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--no-warnings`
+
+Skip global warnings, show command output only
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+## talk:monitor:calls
+
+Prints a list with conversations that have an active call as well as their participant count
+
+### Usage
+
+* `talk:monitor:calls [--output [OUTPUT]]`
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:active-calls
+
+Allows you to check if calls are currently in process
+
+### Usage
+
+* `talk:active-calls [--output [OUTPUT]]`
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:monitor:room
+
+Prints a list with conversations that have an active call as well as their participant count
+
+### Usage
+
+* `talk:monitor:room [--output [OUTPUT]] [--separator SEPARATOR] [--] <token>`
+
+### Arguments
+
+#### `token`
+
+Token of the room to monitor
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+#### `--separator`
+
+Separator for the CSV list when output=csv is used
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `','`
+
+## talk:room:add
+
+Adds users to a room
+
+### Usage
+
+* `talk:room:add [--user USER] [--group GROUP] [--] <token>`
+
+### Arguments
+
+#### `token`
+
+Token of the room to add users to
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--user`
+
+Invites the given users to the room
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+#### `--group`
+
+Invites all members of the given groups to the room
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+## talk:room:create
+
+Create a new room
+
+### Usage
+
+* `talk:room:create [--description DESCRIPTION] [--user USER] [--group GROUP] [--public] [--readonly] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--moderator MODERATOR] [--message-expiration MESSAGE-EXPIRATION] [--] <name>`
+
+### Arguments
+
+#### `name`
+
+The name of the room to create
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--description`
+
+The description of the room to create
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--user`
+
+Invites the given users to the room to create
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+#### `--group`
+
+Invites all members of the given group to the room to create
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+#### `--public`
+
+Creates the room as public room if set
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--readonly`
+
+Creates the room with read-only access only if set
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--listable`
+
+Creates the room with the given listable scope
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--password`
+
+Protects the room to create with the given password
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--owner`
+
+Sets the given user as owner of the room to create
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--moderator`
+
+Promotes the given users to moderators
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+#### `--message-expiration`
+
+Seconds to expire a message after sent. If zero will disable the expire message duration.
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+## talk:room:delete
+
+Deletes a room
+
+### Usage
+
+* `talk:room:delete <token>`
+
+### Arguments
+
+#### `token`
+
+Token of the room to delete
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:room:demote
+
+Demotes participants of a room to regular users
+
+### Usage
+
+* `talk:room:demote <token> <participant>...`
+
+### Arguments
+
+#### `token`
+
+Token of the room in which users should be demoted
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `participant`
+
+Demotes the given participants of the room to regular users
+
+* Is required: yes
+* Is array: yes
+* Default: `array ()`
+
+## talk:room:promote
+
+Promotes participants of a room to moderators
+
+### Usage
+
+* `talk:room:promote <token> <participant>...`
+
+### Arguments
+
+#### `token`
+
+Token of the room in which users should be promoted
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `participant`
+
+Promotes the given participants of the room to moderators
+
+* Is required: yes
+* Is array: yes
+* Default: `array ()`
+
+## talk:room:remove
+
+Remove users from a room
+
+### Usage
+
+* `talk:room:remove <token> <participant>...`
+
+### Arguments
+
+#### `token`
+
+Token of the room to remove users from
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `participant`
+
+Removes the given participants from the room
+
+* Is required: yes
+* Is array: yes
+* Default: `array ()`
+
+## talk:room:update
+
+Updates a room
+
+### Usage
+
+* `talk:room:update [--name NAME] [--description DESCRIPTION] [--public PUBLIC] [--readonly READONLY] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--message-expiration MESSAGE-EXPIRATION] [--] <token>`
+
+### Arguments
+
+#### `token`
+
+The token of the room to update
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--name`
+
+Sets a new name for the room
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--description`
+
+Sets a new description for the room
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--public`
+
+Modifies the room to be a public room (value 1) or private room (value 0)
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--readonly`
+
+Modifies the room to be read-only (value 1) or read-write (value 0)
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--listable`
+
+Modifies the room's listable scope
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--password`
+
+Sets a new password for the room; pass an empty value to remove password protection
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--owner`
+
+Sets the given user as owner of the room; pass an empty value to remove the owner
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--message-expiration`
+
+Seconds to expire a message after sent. If zero will disable the expire message duration.
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+## talk:signaling:add
+
+Add an external signaling server.
+
+### Usage
+
+* `talk:signaling:add [--verify] [--] <server> <secret>`
+
+### Arguments
+
+#### `server`
+
+A server string, ex. wss://signaling.example.org
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `secret`
+
+A shared secret string.
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--verify`
+
+Validate SSL certificate if set.
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+## talk:signaling:delete
+
+Remove an existing signaling server.
+
+### Usage
+
+* `talk:signaling:delete <server>`
+
+### Arguments
+
+#### `server`
+
+An external signaling server string, ex. wss://signaling.example.org
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:signaling:list
+
+List external signaling servers.
+
+### Usage
+
+* `talk:signaling:list [--output [OUTPUT]]`
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:stun:add
+
+Add a new STUN server.
+
+### Usage
+
+* `talk:stun:add <server>`
+
+### Arguments
+
+#### `server`
+
+A domain name and port number separated by the colons, ex. stun.nextcloud.com:443
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:stun:delete
+
+Remove an existing STUN server.
+
+### Usage
+
+* `talk:stun:delete <server>`
+
+### Arguments
+
+#### `server`
+
+A domain name and port number separated by the colons, ex. stun.nextcloud.com:443
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:stun:list
+
+List STUN servers.
+
+### Usage
+
+* `talk:stun:list [--output [OUTPUT]]`
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:turn:add
+
+Add a TURN server.
+
+### Usage
+
+* `talk:turn:add [--secret SECRET] [--generate-secret] [--] <schemes> <server> <protocols>`
+
+### Arguments
+
+#### `schemes`
+
+Schemes, can be turn or turns or turn,turns.
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `server`
+
+A domain name, ex. turn.nextcloud.com
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `protocols`
+
+Protocols, can be udp or tcp or udp,tcp.
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--secret`
+
+A shard secret string
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: no
+* Is negatable: no
+* Default: `NULL`
+
+#### `--generate-secret`
+
+Generate secret if set.
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+## talk:turn:delete
+
+Remove an existing TURN server.
+
+### Usage
+
+* `talk:turn:delete <schemes> <server> <protocols>`
+
+### Arguments
+
+#### `schemes`
+
+Schemes, can be turn or turns or turn,turns
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `server`
+
+A domain name, ex. turn.nextcloud.com
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `protocols`
+
+Protocols, can be udp or tcp or udp,tcp
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+## talk:turn:list
+
+List TURN servers.
+
+### Usage
+
+* `talk:turn:list [--output [OUTPUT]]`
+### Options
+
+#### `--output`
+
+Output format (plain, json or json_pretty, default is plain)
+
+* Accept value: yes
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `'plain'`
+
+## talk:user:remove
+
+Remove a user from all their rooms
+
+### Usage
+
+* `talk:user:remove [--user USER]`
+### Options
+
+#### `--user`
+
+Remove the given users from all rooms
+
+* Accept value: yes
+* Is value required: yes
+* Is multiple: yes
+* Is negatable: no
+* Default: `array ()`
+
+## talk:user:transfer-ownership
+
+Adds the destination-user with the same participant type to all (not one-to-one) conversations of source-user
+
+### Usage
+
+* `talk:user:transfer-ownership [--include-non-moderator] [--remove-source-user] [--] <source-user> <destination-user>`
+
+### Arguments
+
+#### `source-user`
+
+Owner of conversations which shall be moved
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+#### `destination-user`
+
+User who will be the new owner of the conversations
+
+* Is required: yes
+* Is array: no
+* Default: `NULL`
+
+### Options
+
+#### `--include-non-moderator`
+
+Also include conversations where the source-user is a normal user
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
+
+#### `--remove-source-user`
+
+Remove the source-user from the conversations
+
+* Accept value: no
+* Is value required: no
+* Is multiple: no
+* Is negatable: no
+* Default: `false`
 
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
new file mode 100644
index 00000000000..e4cc63829f2
--- /dev/null
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -0,0 +1,168 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+namespace OCA\Talk\Command\Developer;
+
+use OC\Core\Command\Base;
+use OCP\App\IAppManager;
+use OCP\IConfig;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\BufferedOutput;
+use Symfony\Component\Console\Output\OutputInterface;
+
+class UpdateDocs extends Base {
+	private IConfig $config;
+	private IAppManager $appManager;
+	/** @var resource */
+	private $doc;
+
+	public function __construct(IConfig $config) {
+		$this->config = $config;
+
+		parent::__construct();
+	}
+
+	public function isEnabled(): bool {
+		return $this->config->getSystemValue('debug', false) === true;
+	}
+
+	protected function configure(): void {
+		parent::configure();
+
+		$this
+			->setName('talk:developer:update-docs')
+			->setDescription('Update documentation of commands')
+		;
+	}
+
+	protected function execute(InputInterface $input, OutputInterface $output): int {
+		$this->appManager = \OC::$server->get(IAppManager::class);
+		$this->doc = fopen(__DIR__ . '/../../../docs/occ.md', 'a');
+		ftruncate($this->doc, 0);
+
+		$info = $this->appManager->getAppInfo('spreed');
+		foreach ($info['commands'] as $commandNamespace) {
+			$path = str_replace('OCA\Talk', '', $commandNamespace);
+			$path = str_replace('\\', '/', $path);
+			$path = __DIR__ . '/../../../lib' . $path . '.php';
+			if (!file_exists($path)) {
+				$output->writeln('<error>The class of the follow namespase is not implemented: ' . $commandNamespace . '</error>');
+				return 1;
+			}
+			$code = file_get_contents($path);
+			preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
+			if (!array_key_exists('command', $matches)) {
+				preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
+				if (!array_key_exists('command', $matches)) {
+					$output->writeln('<error>A command need to have a name. Namespace: ' . $commandNamespace . '</error>');
+					return 1;
+				}
+			}
+			$command = $matches['command'];
+			$this->updateDocumentation($commandNamespace, $command);
+		}
+		fclose($this->doc);
+		return 0;
+	}
+
+	protected function updateDocumentation(string $namespace, string $commandName): void {
+		$output = new BufferedOutput();
+
+		$command = \OC::$server->get($namespace);
+
+		$output->write('## ' . $command->getName() . "\n\n");
+		$output->write($command->getDescription() . "\n\n");
+		$output->write(
+			'### Usage' . "\n\n" .
+			array_reduce(array_merge([$command->getSynopsis()], $command->getAliases(), $command->getUsages()), function ($carry, $usage) {
+				return $carry.'* `'.$usage.'`'."\n";
+			})
+		);
+		$this->describeInputDefinition($command, $output);
+		$output->write("\n\n");
+
+		fwrite($this->doc, $output->fetch());
+		return;
+	}
+
+	protected function describeInputDefinition(Command $command, OutputInterface $output): void {
+		$definition = $command->getDefinition();
+		if ($showArguments = \count($definition->getArguments()) > 0) {
+			$output->write("\n");
+			$output->write('### Arguments');
+			foreach ($definition->getArguments() as $argument) {
+				$output->write("\n\n");
+				if (null !== $describeInputArgument = $this->describeInputArgument($argument, $output)) {
+					$output->write($describeInputArgument);
+				}
+			}
+		}
+
+		if (\count($definition->getOptions()) > 0) {
+			if ($showArguments) {
+				$output->write("\n\n");
+			}
+
+			$output->write('### Options');
+			foreach ($definition->getOptions() as $option) {
+				$output->write("\n\n");
+				if (null !== $describeInputOption = $this->describeInputOption($option, $output)) {
+					$output->write($describeInputOption);
+				}
+			}
+		}
+	}
+
+	protected function describeInputArgument(InputArgument $argument, OutputInterface $output): void {
+		$output->write(
+			'#### `'.($argument->getName() ?: '<none>')."`\n\n"
+			.($argument->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $argument->getDescription())."\n\n" : '')
+			.'* Is required: '.($argument->isRequired() ? 'yes' : 'no')."\n"
+			.'* Is array: '.($argument->isArray() ? 'yes' : 'no')."\n"
+			.'* Default: `'.str_replace("\n", '', var_export($argument->getDefault(), true)).'`'
+		);
+	}
+
+	protected function describeInputOption(InputOption $option, OutputInterface $output) {
+		$name = '--'.$option->getName();
+		if ($option->isNegatable()) {
+			$name .= '|--no-'.$option->getName();
+		}
+		if ($option->getShortcut()) {
+			$name .= '|-'.str_replace('|', '|-', $option->getShortcut()).'';
+		}
+
+		$output->write(
+			'#### `'.$name.'`'."\n\n"
+			.($option->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $option->getDescription())."\n\n" : '')
+			.'* Accept value: '.($option->acceptValue() ? 'yes' : 'no')."\n"
+			.'* Is value required: '.($option->isValueRequired() ? 'yes' : 'no')."\n"
+			.'* Is multiple: '.($option->isArray() ? 'yes' : 'no')."\n"
+			.'* Is negatable: '.($option->isNegatable() ? 'yes' : 'no')."\n"
+			.'* Default: `'.str_replace("\n", '', var_export($option->getDefault(), true)).'`'
+		);
+	}
+}
diff --git a/tests/php/Command/DocumentationTest.php b/tests/php/Command/DocumentationTest.php
index 462d2375762..669cfb86bd0 100644
--- a/tests/php/Command/DocumentationTest.php
+++ b/tests/php/Command/DocumentationTest.php
@@ -47,6 +47,10 @@ public function testCommandIsDocumented(): void {
 
 	public function commandIsDocummented(string $command): void {
 		$docs = file_get_contents(__DIR__ . '/../../../docs/occ.md');
-		self::assertStringContainsString($command, $docs, 'Asserting that command ' . $command . ' is documented');
+		self::assertStringContainsString(
+			$command,
+			$docs,
+			'The command ' . $command . " haven't documentation. Run the command talk:developer:update-docs."
+		);
 	}
 }
diff --git a/tests/psalm-baseline.xml b/tests/psalm-baseline.xml
index 6a525357c67..30c6693db14 100644
--- a/tests/psalm-baseline.xml
+++ b/tests/psalm-baseline.xml
@@ -1,198 +1,198 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.2.0@fb685a16df3050d4c18d8a4100fe83abe6458cba">
+<files psalm-version="5.6.0@e784128902dfe01d489c4123d69918a9f3c1eac5">
   <file src="lib/AppInfo/Application.php">
-    <UndefinedClass occurrences="2">
+    <UndefinedClass>
       <code>BeforeTemplateRenderedEvent</code>
       <code>BeforeTemplateRenderedEvent</code>
     </UndefinedClass>
-    <UndefinedInterfaceMethod occurrences="1">
+    <UndefinedInterfaceMethod>
       <code>getSettingsManager</code>
     </UndefinedInterfaceMethod>
   </file>
   <file src="lib/BackgroundJob/CheckReferenceIdColumn.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>SchemaWrapper</code>
     </UndefinedClass>
   </file>
   <file src="lib/Chat/ChatManager.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>NullCache</code>
     </UndefinedClass>
   </file>
   <file src="lib/Chat/MessageParser.php">
-    <UndefinedVariable occurrences="1">
+    <UndefinedVariable>
       <code>$guestNames</code>
     </UndefinedVariable>
   </file>
   <file src="lib/Chat/Parser/SystemMessage.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>\OC_Util</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Command/Add.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Command/AddSamples.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Command/Delete.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Command/ListCommand.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Command/Update.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
+      <code>Base</code>
+    </UndefinedClass>
+  </file>
+  <file src="lib/Command/Developer/UpdateDocs.php">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Monitor/Calls.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Monitor/HasActiveCalls.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Monitor/Room.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Add.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Create.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Delete.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Demote.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Promote.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Remove.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Room/Update.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Signaling/Add.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Signaling/Delete.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Signaling/ListCommand.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Stun/Add.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Stun/Delete.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Stun/ListCommand.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Turn/Add.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Turn/Delete.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/Turn/ListCommand.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/User/Remove.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Command/User/TransferOwnership.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Base</code>
     </UndefinedClass>
   </file>
   <file src="lib/Config.php">
-    <NullArgument occurrences="1">
+    <NullArgument>
       <code>null</code>
     </NullArgument>
   </file>
-  <file src="lib/Controller/ChatController.php">
-    <InvalidArrayOffset occurrences="1">
-      <code>$commentIdToIndex[$parentId]</code>
-    </InvalidArrayOffset>
-  </file>
   <file src="lib/Controller/SettingsController.php">
-    <InvalidArgument occurrences="1">
+    <InvalidArgument>
       <code>SharedStorage::class</code>
     </InvalidArgument>
   </file>
   <file src="lib/Controller/TempAvatarController.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Filesystem</code>
     </UndefinedClass>
   </file>
   <file src="lib/Files/TemplateLoader.php">
-    <UndefinedClass occurrences="2">
+    <UndefinedClass>
       <code>LoadSidebar</code>
       <code>LoadSidebar</code>
     </UndefinedClass>
   </file>
   <file src="lib/Files/Util.php">
-    <InvalidArgument occurrences="2">
+    <InvalidArgument>
       <code>$fileId</code>
       <code>SharedStorage::class</code>
     </InvalidArgument>
   </file>
   <file src="lib/MatterbridgeManager.php">
-    <UndefinedClass occurrences="7">
+    <UndefinedClass>
       <code>$this-&gt;tokenProvider</code>
       <code>$this-&gt;tokenProvider</code>
       <code>$this-&gt;tokenProvider</code>
@@ -203,49 +203,48 @@
     </UndefinedClass>
   </file>
   <file src="lib/Middleware/InjectionMiddleware.php">
-    <UndefinedInterfaceMethod occurrences="1">
+    <UndefinedInterfaceMethod>
       <code>getAnnotationParameter</code>
     </UndefinedInterfaceMethod>
   </file>
   <file src="lib/Migration/Version2001Date20170707115443.php">
-    <InvalidArrayAccess occurrences="1">
+    <InvalidArrayAccess>
       <code>$return['num_rooms']</code>
     </InvalidArrayAccess>
   </file>
   <file src="lib/Notification/Notifier.php">
-    <InvalidPropertyAssignmentValue occurrences="3">
+    <InvalidPropertyAssignmentValue>
       <code>$this-&gt;participants</code>
       <code>$this-&gt;rooms</code>
       <code>$this-&gt;rooms</code>
     </InvalidPropertyAssignmentValue>
   </file>
   <file src="lib/PublicShare/TemplateLoader.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>BeforeTemplateRenderedEvent</code>
     </UndefinedClass>
   </file>
   <file src="lib/PublicShareAuth/TemplateLoader.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>BeforeTemplateRenderedEvent</code>
     </UndefinedClass>
   </file>
   <file src="lib/Service/AvatarService.php">
-    <UndefinedClass occurrences="2">
+    <UndefinedClass>
       <code>Filesystem</code>
       <code>\OC_Image</code>
     </UndefinedClass>
   </file>
   <file src="lib/Service/RecordingService.php">
-    <UndefinedClass occurrences="2">
-      <code>Filesystem</code>
+    <UndefinedClass>
       <code>NoUserException</code>
     </UndefinedClass>
   </file>
   <file src="lib/Share/Listener.php">
-    <InvalidArgument occurrences="1">
+    <InvalidArgument>
       <code>[self::class, 'listenPreShare']</code>
     </InvalidArgument>
-    <UndefinedClass occurrences="7">
+    <UndefinedClass>
       <code>$event-&gt;getView()</code>
       <code>$event-&gt;getView()</code>
       <code>$event-&gt;getView()</code>
@@ -256,12 +255,12 @@
     </UndefinedClass>
   </file>
   <file src="lib/Share/RoomShareProvider.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>Cache</code>
     </UndefinedClass>
   </file>
   <file src="lib/TInitialState.php">
-    <UndefinedClass occurrences="1">
+    <UndefinedClass>
       <code>NoUserException</code>
     </UndefinedClass>
   </file>

From 8cd1786865c2bcb472a4f9eb7bbaa60763141bab Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Wed, 1 Mar 2023 22:17:57 -0300
Subject: [PATCH 03/12] Add summary

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          | 29 ++++++++++++++++++
 lib/Command/Developer/UpdateDocs.php | 46 ++++++++++++++++------------
 2 files changed, 56 insertions(+), 19 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index 07379b86540..63c6aee51a7 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -1,3 +1,32 @@
+# Talk occ commands
+
+ * [talk:command:add](#talkcommandadd)
+ * [talk:command:add-samples](#talkcommandadd-samples)
+ * [talk:command:delete](#talkcommanddelete)
+ * [talk:command:list](#talkcommandlist)
+ * [talk:command:update](#talkcommandupdate)
+ * [talk:developer:update-docs](#talkdeveloperupdate-docs)
+ * [talk:monitor:calls](#talkmonitorcalls)
+ * [talk:active-calls](#talkactive-calls)
+ * [talk:monitor:room](#talkmonitorroom)
+ * [talk:room:add](#talkroomadd)
+ * [talk:room:create](#talkroomcreate)
+ * [talk:room:delete](#talkroomdelete)
+ * [talk:room:demote](#talkroomdemote)
+ * [talk:room:promote](#talkroompromote)
+ * [talk:room:remove](#talkroomremove)
+ * [talk:room:update](#talkroomupdate)
+ * [talk:signaling:add](#talksignalingadd)
+ * [talk:signaling:delete](#talksignalingdelete)
+ * [talk:signaling:list](#talksignalinglist)
+ * [talk:stun:add](#talkstunadd)
+ * [talk:stun:delete](#talkstundelete)
+ * [talk:stun:list](#talkstunlist)
+ * [talk:turn:add](#talkturnadd)
+ * [talk:turn:delete](#talkturndelete)
+ * [talk:turn:list](#talkturnlist)
+ * [talk:user:remove](#talkuserremove)
+ * [talk:user:transfer-ownership](#talkusertransfer-ownership)
 ## talk:command:add
 
 Add a new command
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index e4cc63829f2..2aeadfa39e5 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -64,30 +64,38 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		ftruncate($this->doc, 0);
 
 		$info = $this->appManager->getAppInfo('spreed');
-		foreach ($info['commands'] as $commandNamespace) {
-			$path = str_replace('OCA\Talk', '', $commandNamespace);
-			$path = str_replace('\\', '/', $path);
-			$path = __DIR__ . '/../../../lib' . $path . '.php';
-			if (!file_exists($path)) {
-				$output->writeln('<error>The class of the follow namespase is not implemented: ' . $commandNamespace . '</error>');
-				return 1;
-			}
-			$code = file_get_contents($path);
-			preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
-			if (!array_key_exists('command', $matches)) {
-				preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
-				if (!array_key_exists('command', $matches)) {
-					$output->writeln('<error>A command need to have a name. Namespace: ' . $commandNamespace . '</error>');
-					return 1;
-				}
-			}
-			$command = $matches['command'];
-			$this->updateDocumentation($commandNamespace, $command);
+		$commands = [];
+		fwrite($this->doc, "# Talk occ commands\n\n");
+		foreach ($info['commands'] as $namespace) {
+			$command = $this->getCommandByNamespace($namespace);
+			$commands[$namespace] = $command;
+			fwrite($this->doc, ' * [' . $command . '](#' . str_replace([':'], '', $command) . ")\n");
+		}
+		foreach ($commands as $namespace => $command) {
+			$this->updateDocumentation($namespace, $command);
 		}
 		fclose($this->doc);
 		return 0;
 	}
 
+	protected function getCommandByNamespace(string $namespace): string {
+		$path = str_replace('OCA\Talk', '', $namespace);
+		$path = str_replace('\\', '/', $path);
+		$path = __DIR__ . '/../../../lib' . $path . '.php';
+		if (!file_exists($path)) {
+			throw new \Exception('The class of the follow namespase is not implemented: ' . $namespace);
+		}
+		$code = file_get_contents($path);
+		preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
+		if (!array_key_exists('command', $matches)) {
+			preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
+			if (!array_key_exists('command', $matches)) {
+				throw new \Exception('A command need to have a name. Namespace: ' . $namespace);
+			}
+		}
+		return $matches['command'];
+	}
+
 	protected function updateDocumentation(string $namespace, string $commandName): void {
 		$output = new BufferedOutput();
 

From 2e15b55e3e8d29374fd96936b5bb593a5ef1304d Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Wed, 1 Mar 2023 23:27:32 -0300
Subject: [PATCH 04/12] Remove --option of update docs command

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          |  94 +-------------------
 lib/Command/Developer/UpdateDocs.php | 123 ++++++++++++---------------
 2 files changed, 56 insertions(+), 161 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index 63c6aee51a7..a8c139f6427 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -194,99 +194,8 @@ Update documentation of commands
 
 ### Usage
 
-* `talk:developer:update-docs [--output [OUTPUT]]`
+* `talk:developer:update-docs`
 
-### Arguments
-
-#### `command`
-
-The command to execute
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--output`
-
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
-
-#### `--help|-h`
-
-Display help for the given command. When no command is given display help for the list command
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--quiet|-q`
-
-Do not output any message
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--verbose|-v|-vv|-vvv`
-
-Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--version|-V`
-
-Display this application version
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--ansi|--no-ansi`
-
-Force (or disable --no-ansi) ANSI output
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: yes
-* Default: `NULL`
-
-#### `--no-interaction|-n`
-
-Do not ask any interactive question
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--no-warnings`
-
-Skip global warnings, show command output only
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
 
 ## talk:monitor:calls
 
@@ -1027,4 +936,3 @@ Remove the source-user from the conversations
 * Is multiple: no
 * Is negatable: no
 * Default: `false`
-
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index 2aeadfa39e5..d57a9b08748 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -30,14 +30,12 @@
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\BufferedOutput;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class UpdateDocs extends Base {
 	private IConfig $config;
 	private IAppManager $appManager;
-	/** @var resource */
-	private $doc;
+	private array $sections = [];
 
 	public function __construct(IConfig $config) {
 		$this->config = $config;
@@ -50,8 +48,6 @@ public function isEnabled(): bool {
 	}
 
 	protected function configure(): void {
-		parent::configure();
-
 		$this
 			->setName('talk:developer:update-docs')
 			->setDescription('Update documentation of commands')
@@ -60,101 +56,93 @@ protected function configure(): void {
 
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$this->appManager = \OC::$server->get(IAppManager::class);
-		$this->doc = fopen(__DIR__ . '/../../../docs/occ.md', 'a');
-		ftruncate($this->doc, 0);
 
 		$info = $this->appManager->getAppInfo('spreed');
-		$commands = [];
-		fwrite($this->doc, "# Talk occ commands\n\n");
+		$documentation = '';
 		foreach ($info['commands'] as $namespace) {
-			$command = $this->getCommandByNamespace($namespace);
-			$commands[$namespace] = $command;
-			fwrite($this->doc, ' * [' . $command . '](#' . str_replace([':'], '', $command) . ")\n");
-		}
-		foreach ($commands as $namespace => $command) {
-			$this->updateDocumentation($namespace, $command);
+			$this->sections['documentation'][] = $this->getDocumentation($namespace);
 		}
-		fclose($this->doc);
+		$documentation =
+			"# Talk occ commands\n\n" .
+			implode("\n", $this->sections['summary']) . "\n\n" .
+			implode("\n", $this->sections['documentation']);
+
+		$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
+		fwrite($handle, $documentation);
+		fclose($handle);
 		return 0;
 	}
 
-	protected function getCommandByNamespace(string $namespace): string {
-		$path = str_replace('OCA\Talk', '', $namespace);
-		$path = str_replace('\\', '/', $path);
-		$path = __DIR__ . '/../../../lib' . $path . '.php';
-		if (!file_exists($path)) {
-			throw new \Exception('The class of the follow namespase is not implemented: ' . $namespace);
-		}
-		$code = file_get_contents($path);
-		preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
-		if (!array_key_exists('command', $matches)) {
-			preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
-			if (!array_key_exists('command', $matches)) {
-				throw new \Exception('A command need to have a name. Namespace: ' . $namespace);
-			}
-		}
-		return $matches['command'];
-	}
-
-	protected function updateDocumentation(string $namespace, string $commandName): void {
-		$output = new BufferedOutput();
-
+	protected function getDocumentation(string $namespace): string {
 		$command = \OC::$server->get($namespace);
+		// Clean full definition of command that have the default Symfony options
+		$command->setApplication($this->getApplication());
+
+		$this->sections['summary'][] =
+			' * ' .
+			'[' . $command->getName() . ']' .
+			'(#' . str_replace([':'], '', $command->getName()) . ')';
 
-		$output->write('## ' . $command->getName() . "\n\n");
-		$output->write($command->getDescription() . "\n\n");
-		$output->write(
+		$doc = '## ' . $command->getName() . "\n\n";
+		$doc .= $command->getDescription() . "\n\n";
+		$doc .=
 			'### Usage' . "\n\n" .
-			array_reduce(array_merge([$command->getSynopsis()], $command->getAliases(), $command->getUsages()), function ($carry, $usage) {
-				return $carry.'* `'.$usage.'`'."\n";
-			})
-		);
-		$this->describeInputDefinition($command, $output);
-		$output->write("\n\n");
-
-		fwrite($this->doc, $output->fetch());
-		return;
+			array_reduce(
+				array_merge(
+					[$command->getSynopsis()],
+					$command->getAliases(),
+					$command->getUsages()
+				),
+				function ($carry, $usage) {
+					return $carry.'* `'.$usage.'`'."\n";
+				}
+			);
+		$doc .= $this->describeInputDefinition($command);
+		$doc .= "\n";
+
+		return $doc;
 	}
 
-	protected function describeInputDefinition(Command $command, OutputInterface $output): void {
+	protected function describeInputDefinition(Command $command): string {
 		$definition = $command->getDefinition();
+		$text = '';
 		if ($showArguments = \count($definition->getArguments()) > 0) {
-			$output->write("\n");
-			$output->write('### Arguments');
+			$text .= "\n";
+			$text .= '### Arguments';
 			foreach ($definition->getArguments() as $argument) {
-				$output->write("\n\n");
-				if (null !== $describeInputArgument = $this->describeInputArgument($argument, $output)) {
-					$output->write($describeInputArgument);
+				$text .= "\n\n";
+				if (null !== $describeInputArgument = $this->describeInputArgument($argument)) {
+					$text .= $describeInputArgument;
 				}
 			}
 		}
 
 		if (\count($definition->getOptions()) > 0) {
 			if ($showArguments) {
-				$output->write("\n\n");
+				$text .= "\n\n";
 			}
 
-			$output->write('### Options');
+			$text .= '### Options';
 			foreach ($definition->getOptions() as $option) {
-				$output->write("\n\n");
-				if (null !== $describeInputOption = $this->describeInputOption($option, $output)) {
-					$output->write($describeInputOption);
+				$text .= "\n\n";
+				if (null !== $describeInputOption = $this->describeInputOption($option)) {
+					$text .= $describeInputOption;
 				}
 			}
 		}
+		return $text;
 	}
 
-	protected function describeInputArgument(InputArgument $argument, OutputInterface $output): void {
-		$output->write(
+	protected function describeInputArgument(InputArgument $argument): string {
+		return
 			'#### `'.($argument->getName() ?: '<none>')."`\n\n"
 			.($argument->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $argument->getDescription())."\n\n" : '')
 			.'* Is required: '.($argument->isRequired() ? 'yes' : 'no')."\n"
 			.'* Is array: '.($argument->isArray() ? 'yes' : 'no')."\n"
-			.'* Default: `'.str_replace("\n", '', var_export($argument->getDefault(), true)).'`'
-		);
+			.'* Default: `'.str_replace("\n", '', var_export($argument->getDefault(), true)).'`';
 	}
 
-	protected function describeInputOption(InputOption $option, OutputInterface $output) {
+	protected function describeInputOption(InputOption $option): string {
 		$name = '--'.$option->getName();
 		if ($option->isNegatable()) {
 			$name .= '|--no-'.$option->getName();
@@ -163,14 +151,13 @@ protected function describeInputOption(InputOption $option, OutputInterface $out
 			$name .= '|-'.str_replace('|', '|-', $option->getShortcut()).'';
 		}
 
-		$output->write(
+		return
 			'#### `'.$name.'`'."\n\n"
 			.($option->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $option->getDescription())."\n\n" : '')
 			.'* Accept value: '.($option->acceptValue() ? 'yes' : 'no')."\n"
 			.'* Is value required: '.($option->isValueRequired() ? 'yes' : 'no')."\n"
 			.'* Is multiple: '.($option->isArray() ? 'yes' : 'no')."\n"
 			.'* Is negatable: '.($option->isNegatable() ? 'yes' : 'no')."\n"
-			.'* Default: `'.str_replace("\n", '', var_export($option->getDefault(), true)).'`'
-		);
+			.'* Default: `'.str_replace("\n", '', var_export($option->getDefault(), true)).'`';
 	}
 }

From 8ca055cc683400635e8825c1e334d19480279905 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 13:04:43 -0300
Subject: [PATCH 05/12] * Convert arguments and options to table format * Add
 the markdown occ to mkdocs * Add GitHub Action to check if the documentation
 is up to date

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 .../workflows/occ-command-documentation.yml   |  91 ++
 docs/occ.md                                   | 785 ++++--------------
 lib/Command/Developer/UpdateDocs.php          |  77 +-
 mkdocs.yml                                    |   1 +
 tests/php/Command/DocumentationTest.php       |  56 --
 5 files changed, 286 insertions(+), 724 deletions(-)
 create mode 100644 .github/workflows/occ-command-documentation.yml
 delete mode 100644 tests/php/Command/DocumentationTest.php

diff --git a/.github/workflows/occ-command-documentation.yml b/.github/workflows/occ-command-documentation.yml
new file mode 100644
index 00000000000..9f439a25ad5
--- /dev/null
+++ b/.github/workflows/occ-command-documentation.yml
@@ -0,0 +1,91 @@
+name: Occ command documentation
+
+on:
+  pull_request:
+    paths:
+      - '.github/workflows/**'
+      - 'appinfo/**'
+      - 'lib/**'
+      - 'templates/**'
+      - 'tests/**'
+      - 'vendor/**'
+      - 'vendor-bin/**'
+      - '.php-cs-fixer.dist.php'
+      - 'composer.json'
+      - 'composer.lock'
+
+  push:
+    branches:
+      - main
+      - master
+      - stable*
+
+permissions:
+  contents: read
+
+concurrency:
+  group: occ-command-documentation-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  occ-command-documentation:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php-versions: ['8.0', '8.1']
+        server-versions: ['master']
+
+    steps:
+      - name: Set app env
+        run: |
+          # Split and keep last
+          echo "APP_NAME=${GITHUB_REPOSITORY##*/}" >> $GITHUB_ENV
+
+      - name: Checkout server
+        uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c # v3
+        with:
+          submodules: true
+          repository: nextcloud/server
+          ref: ${{ matrix.server-versions }}
+
+      - name: Checkout app
+        uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c # v3
+        with:
+          path: apps/${{ env.APP_NAME }}
+
+      - name: Set up php ${{ matrix.php-versions }}
+        uses: shivammathur/setup-php@1a18b2267f80291a81ca1d33e7c851fe09e7dfc4 # v2
+        with:
+          php-version: ${{ matrix.php-versions }}
+          extensions: mbstring, iconv, fileinfo, intl, sqlite, pdo_sqlite
+          coverage: none
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Check composer file existence
+        id: check_composer
+        uses: andstor/file-existence-action@20b4d2e596410855db8f9ca21e96fbe18e12930b # v2
+        with:
+          files: apps/${{ env.APP_NAME }}/composer.json
+
+      - name: Set up dependencies
+        # Only run if phpunit config file exists
+        if: steps.check_composer.outputs.files_exists == 'true'
+        working-directory: apps/${{ env.APP_NAME }}
+        run: composer i
+
+      - name: Set up Nextcloud
+        env:
+          DB_PORT: 4444
+        run: |
+          mkdir data
+          ./occ maintenance:install --verbose --database=sqlite --database-name=nextcloud --database-host=127.0.0.1 --database-port=$DB_PORT --database-user=root --database-pass=rootpassword --admin-user admin --admin-pass password
+          ./occ app:enable --force ${{ env.APP_NAME }}
+          ./occ config:system:set debug --value true --type boolean
+
+      - name: Check if documentation is up to date
+        id: check_documentation
+        run: |
+          ./occ talk:developer:update-docs
+          bash -c "[[ ! \"`git status --porcelain `\" ]] || (echo 'Documentation outdated. Run the command talk:developer:update-docs locally, commit the occ.md file and push the changes.' && exit 1)"
diff --git a/docs/occ.md b/docs/occ.md
index a8c139f6427..0518f6ffbbc 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -27,6 +27,7 @@
  * [talk:turn:list](#talkturnlist)
  * [talk:user:remove](#talkuserremove)
  * [talk:user:transfer-ownership](#talkusertransfer-ownership)
+
 ## talk:command:add
 
 Add a new command
@@ -35,47 +36,14 @@ Add a new command
 
 * `talk:command:add <cmd> <name> <script> <response> <enabled>`
 
-### Arguments
-
-#### `cmd`
-
-The command as used in the chat "/help" => "help"
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `name`
-
-Name of the user posting the response
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `script`
-
-Script to execute (Must be using absolute paths only)
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `response`
-
-Who should see the response: 0 - No one, 1 - User, 2 - All
 
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `enabled`
-
-Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `cmd` | The command as used in the chat "/help" => "help" | yes | no | `NULL` |
+| `name` | Name of the user posting the response | yes | no | `NULL` |
+| `script` | Script to execute (Must be using absolute paths only) | yes | no | `NULL` |
+| `response` | Who should see the response: 0 - No one, 1 - User, 2 - All | yes | no | `NULL` |
+| `enabled` | Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests | yes | no | `NULL` |
 
 ## talk:command:add-samples
 
@@ -86,6 +54,7 @@ Adds some sample commands: /wiki, …
 * `talk:command:add-samples`
 
 
+
 ## talk:command:delete
 
 Remove an existing command
@@ -94,13 +63,10 @@ Remove an existing command
 
 * `talk:command:delete <command-id>`
 
-### Arguments
-
-#### `command-id`
 
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `command-id` |  | yes | no | `NULL` |
 
 ## talk:command:list
 
@@ -110,27 +76,14 @@ List all available commands
 
 * `talk:command:list [--output [OUTPUT]] [--] [<app>]`
 
-### Arguments
-
-#### `app`
 
-Only list the commands of a specific app, "custom" to list all custom commands
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `app` | Only list the commands of a specific app, "custom" to list all custom commands | no | no | `NULL` |
 
-* Is required: no
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--output`
-
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:command:update
 
@@ -140,53 +93,15 @@ Add a new command
 
 * `talk:command:update <command-id> <cmd> <name> <script> <response> <enabled>`
 
-### Arguments
-
-#### `command-id`
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
 
-#### `cmd`
-
-The command as used in the chat "/help" => "help"
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `name`
-
-Name of the user posting the response
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `script`
-
-Script to execute (Must be using absolute paths only)
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `response`
-
-Who should see the response: 0 - No one, 1 - User, 2 - All
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `enabled`
-
-Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `command-id` |  | yes | no | `NULL` |
+| `cmd` | The command as used in the chat "/help" => "help" | yes | no | `NULL` |
+| `name` | Name of the user posting the response | yes | no | `NULL` |
+| `script` | Script to execute (Must be using absolute paths only) | yes | no | `NULL` |
+| `response` | Who should see the response: 0 - No one, 1 - User, 2 - All | yes | no | `NULL` |
+| `enabled` | Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests | yes | no | `NULL` |
 
 ## talk:developer:update-docs
 
@@ -194,8 +109,12 @@ Update documentation of commands
 
 ### Usage
 
-* `talk:developer:update-docs`
+* `talk:developer:update-docs [<name>]`
+
 
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `name` | The name of command to update. When have this argument, don't will upate the documentation, only will print the markdown output. | no | no | `NULL` |
 
 ## talk:monitor:calls
 
@@ -204,17 +123,10 @@ Prints a list with conversations that have an active call as well as their parti
 ### Usage
 
 * `talk:monitor:calls [--output [OUTPUT]]`
-### Options
 
-#### `--output`
-
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:active-calls
 
@@ -223,17 +135,10 @@ Allows you to check if calls are currently in process
 ### Usage
 
 * `talk:active-calls [--output [OUTPUT]]`
-### Options
-
-#### `--output`
 
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:monitor:room
 
@@ -243,37 +148,15 @@ Prints a list with conversations that have an active call as well as their parti
 
 * `talk:monitor:room [--output [OUTPUT]] [--separator SEPARATOR] [--] <token>`
 
-### Arguments
-
-#### `token`
-
-Token of the room to monitor
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--output`
 
-Output format (plain, json or json_pretty, default is plain)
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room to monitor | yes | no | `NULL` |
 
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
-
-#### `--separator`
-
-Separator for the CSV list when output=csv is used
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `','`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--separator` | Separator for the CSV list when output=csv is used | yes | yes | no | no | ','` |
 
 ## talk:room:add
 
@@ -283,37 +166,15 @@ Adds users to a room
 
 * `talk:room:add [--user USER] [--group GROUP] [--] <token>`
 
-### Arguments
-
-#### `token`
-
-Token of the room to add users to
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
 
-### Options
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room to add users to | yes | no | `NULL` |
 
-#### `--user`
-
-Invites the given users to the room
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
-
-#### `--group`
-
-Invites all members of the given groups to the room
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--user` | Invites the given users to the room | yes | yes | yes | no | array ()` |
+| `--group` | Invites all members of the given groups to the room | yes | yes | yes | no | array ()` |
 
 ## talk:room:create
 
@@ -323,117 +184,23 @@ Create a new room
 
 * `talk:room:create [--description DESCRIPTION] [--user USER] [--group GROUP] [--public] [--readonly] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--moderator MODERATOR] [--message-expiration MESSAGE-EXPIRATION] [--] <name>`
 
-### Arguments
-
-#### `name`
-
-The name of the room to create
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--description`
-
-The description of the room to create
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--user`
-
-Invites the given users to the room to create
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
-
-#### `--group`
-
-Invites all members of the given group to the room to create
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
 
-#### `--public`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `name` | The name of the room to create | yes | no | `NULL` |
 
-Creates the room as public room if set
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--readonly`
-
-Creates the room with read-only access only if set
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--listable`
-
-Creates the room with the given listable scope
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--password`
-
-Protects the room to create with the given password
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--owner`
-
-Sets the given user as owner of the room to create
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--moderator`
-
-Promotes the given users to moderators
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
-
-#### `--message-expiration`
-
-Seconds to expire a message after sent. If zero will disable the expire message duration.
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--description` | The description of the room to create | yes | yes | no | no | NULL` |
+| `--user` | Invites the given users to the room to create | yes | yes | yes | no | array ()` |
+| `--group` | Invites all members of the given group to the room to create | yes | yes | yes | no | array ()` |
+| `--public` | Creates the room as public room if set | no | no | no | no | false` |
+| `--readonly` | Creates the room with read-only access only if set | no | no | no | no | false` |
+| `--listable` | Creates the room with the given listable scope | yes | yes | no | no | NULL` |
+| `--password` | Protects the room to create with the given password | yes | yes | no | no | NULL` |
+| `--owner` | Sets the given user as owner of the room to create | yes | yes | no | no | NULL` |
+| `--moderator` | Promotes the given users to moderators | yes | yes | yes | no | array ()` |
+| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | no | NULL` |
 
 ## talk:room:delete
 
@@ -443,15 +210,10 @@ Deletes a room
 
 * `talk:room:delete <token>`
 
-### Arguments
-
-#### `token`
-
-Token of the room to delete
 
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room to delete | yes | no | `NULL` |
 
 ## talk:room:demote
 
@@ -461,23 +223,11 @@ Demotes participants of a room to regular users
 
 * `talk:room:demote <token> <participant>...`
 
-### Arguments
-
-#### `token`
-
-Token of the room in which users should be demoted
 
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `participant`
-
-Demotes the given participants of the room to regular users
-
-* Is required: yes
-* Is array: yes
-* Default: `array ()`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room in which users should be demoted | yes | no | `NULL` |
+| `participant` | Demotes the given participants of the room to regular users | yes | yes | `array ()` |
 
 ## talk:room:promote
 
@@ -487,23 +237,11 @@ Promotes participants of a room to moderators
 
 * `talk:room:promote <token> <participant>...`
 
-### Arguments
-
-#### `token`
-
-Token of the room in which users should be promoted
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
 
-#### `participant`
-
-Promotes the given participants of the room to moderators
-
-* Is required: yes
-* Is array: yes
-* Default: `array ()`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room in which users should be promoted | yes | no | `NULL` |
+| `participant` | Promotes the given participants of the room to moderators | yes | yes | `array ()` |
 
 ## talk:room:remove
 
@@ -513,23 +251,11 @@ Remove users from a room
 
 * `talk:room:remove <token> <participant>...`
 
-### Arguments
-
-#### `token`
-
-Token of the room to remove users from
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `participant`
 
-Removes the given participants from the room
-
-* Is required: yes
-* Is array: yes
-* Default: `array ()`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | Token of the room to remove users from | yes | no | `NULL` |
+| `participant` | Removes the given participants from the room | yes | yes | `array ()` |
 
 ## talk:room:update
 
@@ -539,97 +265,21 @@ Updates a room
 
 * `talk:room:update [--name NAME] [--description DESCRIPTION] [--public PUBLIC] [--readonly READONLY] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--message-expiration MESSAGE-EXPIRATION] [--] <token>`
 
-### Arguments
-
-#### `token`
-
-The token of the room to update
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--name`
-
-Sets a new name for the room
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--description`
-
-Sets a new description for the room
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--public`
 
-Modifies the room to be a public room (value 1) or private room (value 0)
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `token` | The token of the room to update | yes | no | `NULL` |
 
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--readonly`
-
-Modifies the room to be read-only (value 1) or read-write (value 0)
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--listable`
-
-Modifies the room's listable scope
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--password`
-
-Sets a new password for the room; pass an empty value to remove password protection
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--owner`
-
-Sets the given user as owner of the room; pass an empty value to remove the owner
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--message-expiration`
-
-Seconds to expire a message after sent. If zero will disable the expire message duration.
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--name` | Sets a new name for the room | yes | yes | no | no | NULL` |
+| `--description` | Sets a new description for the room | yes | yes | no | no | NULL` |
+| `--public` | Modifies the room to be a public room (value 1) or private room (value 0) | yes | yes | no | no | NULL` |
+| `--readonly` | Modifies the room to be read-only (value 1) or read-write (value 0) | yes | yes | no | no | NULL` |
+| `--listable` | Modifies the room's listable scope | yes | yes | no | no | NULL` |
+| `--password` | Sets a new password for the room; pass an empty value to remove password protection | yes | yes | no | no | NULL` |
+| `--owner` | Sets the given user as owner of the room; pass an empty value to remove the owner | yes | yes | no | no | NULL` |
+| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | no | NULL` |
 
 ## talk:signaling:add
 
@@ -639,35 +289,15 @@ Add an external signaling server.
 
 * `talk:signaling:add [--verify] [--] <server> <secret>`
 
-### Arguments
-
-#### `server`
-
-A server string, ex. wss://signaling.example.org
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `secret`
-
-A shared secret string.
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--verify`
 
-Validate SSL certificate if set.
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `server` | A server string, ex. wss://signaling.example.org | yes | no | `NULL` |
+| `secret` | A shared secret string. | yes | no | `NULL` |
 
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--verify` | Validate SSL certificate if set. | no | no | no | no | false` |
 
 ## talk:signaling:delete
 
@@ -677,15 +307,10 @@ Remove an existing signaling server.
 
 * `talk:signaling:delete <server>`
 
-### Arguments
 
-#### `server`
-
-An external signaling server string, ex. wss://signaling.example.org
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `server` | An external signaling server string, ex. wss://signaling.example.org | yes | no | `NULL` |
 
 ## talk:signaling:list
 
@@ -694,17 +319,10 @@ List external signaling servers.
 ### Usage
 
 * `talk:signaling:list [--output [OUTPUT]]`
-### Options
-
-#### `--output`
 
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:stun:add
 
@@ -714,15 +332,10 @@ Add a new STUN server.
 
 * `talk:stun:add <server>`
 
-### Arguments
-
-#### `server`
-
-A domain name and port number separated by the colons, ex. stun.nextcloud.com:443
 
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `server` | A domain name and port number separated by the colons, ex. stun.nextcloud.com:443 | yes | no | `NULL` |
 
 ## talk:stun:delete
 
@@ -732,15 +345,10 @@ Remove an existing STUN server.
 
 * `talk:stun:delete <server>`
 
-### Arguments
 
-#### `server`
-
-A domain name and port number separated by the colons, ex. stun.nextcloud.com:443
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `server` | A domain name and port number separated by the colons, ex. stun.nextcloud.com:443 | yes | no | `NULL` |
 
 ## talk:stun:list
 
@@ -749,17 +357,10 @@ List STUN servers.
 ### Usage
 
 * `talk:stun:list [--output [OUTPUT]]`
-### Options
-
-#### `--output`
 
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:turn:add
 
@@ -769,53 +370,17 @@ Add a TURN server.
 
 * `talk:turn:add [--secret SECRET] [--generate-secret] [--] <schemes> <server> <protocols>`
 
-### Arguments
-
-#### `schemes`
-
-Schemes, can be turn or turns or turn,turns.
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `server`
-
-A domain name, ex. turn.nextcloud.com
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `protocols`
-
-Protocols, can be udp or tcp or udp,tcp.
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--secret`
-
-A shard secret string
-
-* Accept value: yes
-* Is value required: yes
-* Is multiple: no
-* Is negatable: no
-* Default: `NULL`
-
-#### `--generate-secret`
 
-Generate secret if set.
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `schemes` | Schemes, can be turn or turns or turn,turns. | yes | no | `NULL` |
+| `server` | A domain name, ex. turn.nextcloud.com | yes | no | `NULL` |
+| `protocols` | Protocols, can be udp or tcp or udp,tcp. | yes | no | `NULL` |
 
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--secret` | A shard secret string | yes | yes | no | no | NULL` |
+| `--generate-secret` | Generate secret if set. | no | no | no | no | false` |
 
 ## talk:turn:delete
 
@@ -825,31 +390,12 @@ Remove an existing TURN server.
 
 * `talk:turn:delete <schemes> <server> <protocols>`
 
-### Arguments
 
-#### `schemes`
-
-Schemes, can be turn or turns or turn,turns
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `server`
-
-A domain name, ex. turn.nextcloud.com
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `protocols`
-
-Protocols, can be udp or tcp or udp,tcp
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `schemes` | Schemes, can be turn or turns or turn,turns | yes | no | `NULL` |
+| `server` | A domain name, ex. turn.nextcloud.com | yes | no | `NULL` |
+| `protocols` | Protocols, can be udp or tcp or udp,tcp | yes | no | `NULL` |
 
 ## talk:turn:list
 
@@ -858,17 +404,10 @@ List TURN servers.
 ### Usage
 
 * `talk:turn:list [--output [OUTPUT]]`
-### Options
-
-#### `--output`
 
-Output format (plain, json or json_pretty, default is plain)
-
-* Accept value: yes
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `'plain'`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
 
 ## talk:user:remove
 
@@ -877,17 +416,10 @@ Remove a user from all their rooms
 ### Usage
 
 * `talk:user:remove [--user USER]`
-### Options
-
-#### `--user`
-
-Remove the given users from all rooms
 
-* Accept value: yes
-* Is value required: yes
-* Is multiple: yes
-* Is negatable: no
-* Default: `array ()`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--user` | Remove the given users from all rooms | yes | yes | yes | no | array ()` |
 
 ## talk:user:transfer-ownership
 
@@ -897,42 +429,13 @@ Adds the destination-user with the same participant type to all (not one-to-one)
 
 * `talk:user:transfer-ownership [--include-non-moderator] [--remove-source-user] [--] <source-user> <destination-user>`
 
-### Arguments
-
-#### `source-user`
-
-Owner of conversations which shall be moved
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-#### `destination-user`
-
-User who will be the new owner of the conversations
-
-* Is required: yes
-* Is array: no
-* Default: `NULL`
-
-### Options
-
-#### `--include-non-moderator`
-
-Also include conversations where the source-user is a normal user
-
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
-
-#### `--remove-source-user`
 
-Remove the source-user from the conversations
+| Arguments | Description | Is required | Is array | Default |
+|---|---|---|---|---|
+| `source-user` | Owner of conversations which shall be moved | yes | no | `NULL` |
+| `destination-user` | User who will be the new owner of the conversations | yes | no | `NULL` |
 
-* Accept value: no
-* Is value required: no
-* Is multiple: no
-* Is negatable: no
-* Default: `false`
+| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+|---|---|---|---|---|---|
+| `--include-non-moderator` | Also include conversations where the source-user is a normal user | no | no | no | no | false` |
+| `--remove-source-user` | Remove the source-user from the conversations | no | no | no | no | false` |
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index d57a9b08748..8ddaa481db7 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -51,6 +51,11 @@ protected function configure(): void {
 		$this
 			->setName('talk:developer:update-docs')
 			->setDescription('Update documentation of commands')
+			->addArgument(
+				'name',
+				InputArgument::OPTIONAL,
+				"The name of command to update. When have this argument, don't will upate the documentation, only will print the markdown output."
+			)
 		;
 	}
 
@@ -58,26 +63,39 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		$this->appManager = \OC::$server->get(IAppManager::class);
 
 		$info = $this->appManager->getAppInfo('spreed');
+		$commandName = $input->getArgument('name');
 		$documentation = '';
 		foreach ($info['commands'] as $namespace) {
-			$this->sections['documentation'][] = $this->getDocumentation($namespace);
+			$command = $this->getCommand($namespace);
+			if ($commandName && $commandName !== $command->getName()) {
+				continue;
+			}
+			$this->sections['documentation'][] = $this->getDocumentation($command);
+		}
+
+		if ($commandName) {
+			$output->writeln(implode("\n", $this->sections['documentation']));
+		} else {
+			$documentation =
+				"# Talk occ commands\n\n" .
+				implode("\n", $this->sections['summary']) . "\n\n" .
+				implode("\n", $this->sections['documentation']);
+
+			$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
+			fwrite($handle, $documentation);
+			fclose($handle);
 		}
-		$documentation =
-			"# Talk occ commands\n\n" .
-			implode("\n", $this->sections['summary']) . "\n\n" .
-			implode("\n", $this->sections['documentation']);
-
-		$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
-		fwrite($handle, $documentation);
-		fclose($handle);
 		return 0;
 	}
 
-	protected function getDocumentation(string $namespace): string {
+	protected function getCommand(string $namespace): Command {
 		$command = \OC::$server->get($namespace);
 		// Clean full definition of command that have the default Symfony options
 		$command->setApplication($this->getApplication());
+		return $command;
+	}
 
+	protected function getDocumentation(Command $command): string {
 		$this->sections['summary'][] =
 			' * ' .
 			'[' . $command->getName() . ']' .
@@ -96,7 +114,7 @@ protected function getDocumentation(string $namespace): string {
 				function ($carry, $usage) {
 					return $carry.'* `'.$usage.'`'."\n";
 				}
-			);
+			) . "\n";
 		$doc .= $this->describeInputDefinition($command);
 		$doc .= "\n";
 
@@ -108,9 +126,10 @@ protected function describeInputDefinition(Command $command): string {
 		$text = '';
 		if ($showArguments = \count($definition->getArguments()) > 0) {
 			$text .= "\n";
-			$text .= '### Arguments';
+			$text .= "| Arguments | Description | Is required | Is array | Default |\n";
+			$text .= '|---|---|---|---|---|';
 			foreach ($definition->getArguments() as $argument) {
-				$text .= "\n\n";
+				$text .= "\n";
 				if (null !== $describeInputArgument = $this->describeInputArgument($argument)) {
 					$text .= $describeInputArgument;
 				}
@@ -122,9 +141,10 @@ protected function describeInputDefinition(Command $command): string {
 				$text .= "\n\n";
 			}
 
-			$text .= '### Options';
+			$text .= "| Options | Accept value | Is value required | Is multiple | is nagatable | Default |\n";
+			$text .= '|---|---|---|---|---|---|';
 			foreach ($definition->getOptions() as $option) {
-				$text .= "\n\n";
+				$text .= "\n";
 				if (null !== $describeInputOption = $this->describeInputOption($option)) {
 					$text .= $describeInputOption;
 				}
@@ -134,12 +154,14 @@ protected function describeInputDefinition(Command $command): string {
 	}
 
 	protected function describeInputArgument(InputArgument $argument): string {
+		$description = $argument->getDescription();
+
 		return
-			'#### `'.($argument->getName() ?: '<none>')."`\n\n"
-			.($argument->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $argument->getDescription())."\n\n" : '')
-			.'* Is required: '.($argument->isRequired() ? 'yes' : 'no')."\n"
-			.'* Is array: '.($argument->isArray() ? 'yes' : 'no')."\n"
-			.'* Default: `'.str_replace("\n", '', var_export($argument->getDefault(), true)).'`';
+			'| `'.($argument->getName() ?: '<none>')."` | " .
+			($description ? preg_replace('/\s*[\r\n]\s*/', " ", $description) : '') . ' | ' .
+			($argument->isRequired() ? 'yes' : 'no')." | " .
+			($argument->isArray() ? 'yes' : 'no')." | " .
+			'`' . str_replace("\n", '', var_export($argument->getDefault(), true)) . "` |";
 	}
 
 	protected function describeInputOption(InputOption $option): string {
@@ -150,14 +172,15 @@ protected function describeInputOption(InputOption $option): string {
 		if ($option->getShortcut()) {
 			$name .= '|-'.str_replace('|', '|-', $option->getShortcut()).'';
 		}
+		$description = $option->getDescription();
 
 		return
-			'#### `'.$name.'`'."\n\n"
-			.($option->getDescription() ? preg_replace('/\s*[\r\n]\s*/', "\n", $option->getDescription())."\n\n" : '')
-			.'* Accept value: '.($option->acceptValue() ? 'yes' : 'no')."\n"
-			.'* Is value required: '.($option->isValueRequired() ? 'yes' : 'no')."\n"
-			.'* Is multiple: '.($option->isArray() ? 'yes' : 'no')."\n"
-			.'* Is negatable: '.($option->isNegatable() ? 'yes' : 'no')."\n"
-			.'* Default: `'.str_replace("\n", '', var_export($option->getDefault(), true)).'`';
+			'| `'.$name.'` | ' .
+			($description ? preg_replace('/\s*[\r\n]\s*/', " ", $description) : '') . ' | '.
+			($option->acceptValue() ? 'yes' : 'no')." | " .
+			($option->isValueRequired() ? 'yes' : 'no')." | " .
+			($option->isArray() ? 'yes' : 'no')." | " .
+			($option->isNegatable() ? 'yes' : 'no')." | " .
+			str_replace("\n", '', var_export($option->getDefault(), true)).'` |';
 	}
 }
diff --git a/mkdocs.yml b/mkdocs.yml
index 734b1913a0d..3a3d4910e1a 100644
--- a/mkdocs.yml
+++ b/mkdocs.yml
@@ -35,6 +35,7 @@ nav:
       - 'Integration by other apps': 'integration.md'
       - 'Signaling API': 'internal-signaling.md'
   - Commands: 'commands.md'
+  - 'Occ commands': 'occ.md'
   - 'PHP events': 'events.md'
   - 'Configuring coTURN': 'TURN.md'
   - 'Matterbridge integration': 'matterbridge.md'
diff --git a/tests/php/Command/DocumentationTest.php b/tests/php/Command/DocumentationTest.php
deleted file mode 100644
index 669cfb86bd0..00000000000
--- a/tests/php/Command/DocumentationTest.php
+++ /dev/null
@@ -1,56 +0,0 @@
-<?php
-/**
- * @copyright Copyright (c) 2023 Vitor Mattos <vitor@php.rio>
- *
- * @author Vitor Mattos <vitor@php.rio>
- *
- * @license GNU AGPL version 3 or any later version
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-
-namespace OCA\Talk\Tests\php\Command;
-
-use OCP\App\IAppManager;
-use Test\TestCase;
-
-final class DocumentationTest extends TestCase {
-	public function testCommandIsDocumented(): void {
-		/** @var IAppManager */
-		$appManager = \OC::$server->get(IAppManager::class);
-		$info = $appManager->getAppInfo('spreed');
-		foreach ($info['commands'] as $commandNamespace) {
-			$path = str_replace('OCA\Talk', '', $commandNamespace);
-			$path = str_replace('\\', '/', $path);
-			$path = __DIR__ . '/../../../lib' . $path . '.php';
-			$this->assertFileExists($path, 'The class of the follow namespase is not implemented: ' . $commandNamespace);
-			$code = file_get_contents($path);
-			preg_match("/->setName\('(?<command>[\w:\-_]+)'\)/", $code, $matches);
-			if (!array_key_exists('command', $matches)) {
-				preg_match("/\tname: '(?<command>[\w:\-_]+)',?\n/", $code, $matches);
-			}
-			$this->assertArrayHasKey('command', $matches, 'A command need to have a name.');
-			$this->commandIsDocummented($matches['command']);
-		}
-	}
-
-	public function commandIsDocummented(string $command): void {
-		$docs = file_get_contents(__DIR__ . '/../../../docs/occ.md');
-		self::assertStringContainsString(
-			$command,
-			$docs,
-			'The command ' . $command . " haven't documentation. Run the command talk:developer:update-docs."
-		);
-	}
-}

From 9f7f4614b4ea3d57f20612d6559c945e8f43a52f Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 22:26:38 -0300
Subject: [PATCH 06/12] Summary removed

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          | 28 ----------------------------
 lib/Command/Developer/UpdateDocs.php |  6 ------
 2 files changed, 34 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index 0518f6ffbbc..37fc603f14f 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -1,33 +1,5 @@
 # Talk occ commands
 
- * [talk:command:add](#talkcommandadd)
- * [talk:command:add-samples](#talkcommandadd-samples)
- * [talk:command:delete](#talkcommanddelete)
- * [talk:command:list](#talkcommandlist)
- * [talk:command:update](#talkcommandupdate)
- * [talk:developer:update-docs](#talkdeveloperupdate-docs)
- * [talk:monitor:calls](#talkmonitorcalls)
- * [talk:active-calls](#talkactive-calls)
- * [talk:monitor:room](#talkmonitorroom)
- * [talk:room:add](#talkroomadd)
- * [talk:room:create](#talkroomcreate)
- * [talk:room:delete](#talkroomdelete)
- * [talk:room:demote](#talkroomdemote)
- * [talk:room:promote](#talkroompromote)
- * [talk:room:remove](#talkroomremove)
- * [talk:room:update](#talkroomupdate)
- * [talk:signaling:add](#talksignalingadd)
- * [talk:signaling:delete](#talksignalingdelete)
- * [talk:signaling:list](#talksignalinglist)
- * [talk:stun:add](#talkstunadd)
- * [talk:stun:delete](#talkstundelete)
- * [talk:stun:list](#talkstunlist)
- * [talk:turn:add](#talkturnadd)
- * [talk:turn:delete](#talkturndelete)
- * [talk:turn:list](#talkturnlist)
- * [talk:user:remove](#talkuserremove)
- * [talk:user:transfer-ownership](#talkusertransfer-ownership)
-
 ## talk:command:add
 
 Add a new command
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index 8ddaa481db7..4762e73f379 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -78,7 +78,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		} else {
 			$documentation =
 				"# Talk occ commands\n\n" .
-				implode("\n", $this->sections['summary']) . "\n\n" .
 				implode("\n", $this->sections['documentation']);
 
 			$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
@@ -96,11 +95,6 @@ protected function getCommand(string $namespace): Command {
 	}
 
 	protected function getDocumentation(Command $command): string {
-		$this->sections['summary'][] =
-			' * ' .
-			'[' . $command->getName() . ']' .
-			'(#' . str_replace([':'], '', $command->getName()) . ')';
-
 		$doc = '## ' . $command->getName() . "\n\n";
 		$doc .= $command->getDescription() . "\n\n";
 		$doc .=

From 248e8b28220dfda0630469b450bdb1f0524afb69 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 22:54:40 -0300
Subject: [PATCH 07/12] Removed suppot to isNegattable because the server
 Symfony Client is an old version

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          | 96 ++++++++++++++--------------
 lib/Command/Developer/UpdateDocs.php |  6 +-
 2 files changed, 49 insertions(+), 53 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index 37fc603f14f..ad1221e5e41 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -53,9 +53,9 @@ List all available commands
 |---|---|---|---|---|
 | `app` | Only list the commands of a specific app, "custom" to list all custom commands | no | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:command:update
 
@@ -96,9 +96,9 @@ Prints a list with conversations that have an active call as well as their parti
 
 * `talk:monitor:calls [--output [OUTPUT]]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:active-calls
 
@@ -108,9 +108,9 @@ Allows you to check if calls are currently in process
 
 * `talk:active-calls [--output [OUTPUT]]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:monitor:room
 
@@ -125,10 +125,10 @@ Prints a list with conversations that have an active call as well as their parti
 |---|---|---|---|---|
 | `token` | Token of the room to monitor | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
-| `--separator` | Separator for the CSV list when output=csv is used | yes | yes | no | no | ','` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
+| `--separator` | Separator for the CSV list when output=csv is used | yes | yes | no | ','` |
 
 ## talk:room:add
 
@@ -143,10 +143,10 @@ Adds users to a room
 |---|---|---|---|---|
 | `token` | Token of the room to add users to | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--user` | Invites the given users to the room | yes | yes | yes | no | array ()` |
-| `--group` | Invites all members of the given groups to the room | yes | yes | yes | no | array ()` |
+| `--user` | Invites the given users to the room | yes | yes | yes | array ()` |
+| `--group` | Invites all members of the given groups to the room | yes | yes | yes | array ()` |
 
 ## talk:room:create
 
@@ -161,18 +161,18 @@ Create a new room
 |---|---|---|---|---|
 | `name` | The name of the room to create | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--description` | The description of the room to create | yes | yes | no | no | NULL` |
-| `--user` | Invites the given users to the room to create | yes | yes | yes | no | array ()` |
-| `--group` | Invites all members of the given group to the room to create | yes | yes | yes | no | array ()` |
-| `--public` | Creates the room as public room if set | no | no | no | no | false` |
-| `--readonly` | Creates the room with read-only access only if set | no | no | no | no | false` |
-| `--listable` | Creates the room with the given listable scope | yes | yes | no | no | NULL` |
-| `--password` | Protects the room to create with the given password | yes | yes | no | no | NULL` |
-| `--owner` | Sets the given user as owner of the room to create | yes | yes | no | no | NULL` |
-| `--moderator` | Promotes the given users to moderators | yes | yes | yes | no | array ()` |
-| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | no | NULL` |
+| `--description` | The description of the room to create | yes | yes | no | NULL` |
+| `--user` | Invites the given users to the room to create | yes | yes | yes | array ()` |
+| `--group` | Invites all members of the given group to the room to create | yes | yes | yes | array ()` |
+| `--public` | Creates the room as public room if set | no | no | no | false` |
+| `--readonly` | Creates the room with read-only access only if set | no | no | no | false` |
+| `--listable` | Creates the room with the given listable scope | yes | yes | no | NULL` |
+| `--password` | Protects the room to create with the given password | yes | yes | no | NULL` |
+| `--owner` | Sets the given user as owner of the room to create | yes | yes | no | NULL` |
+| `--moderator` | Promotes the given users to moderators | yes | yes | yes | array ()` |
+| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | NULL` |
 
 ## talk:room:delete
 
@@ -242,16 +242,16 @@ Updates a room
 |---|---|---|---|---|
 | `token` | The token of the room to update | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--name` | Sets a new name for the room | yes | yes | no | no | NULL` |
-| `--description` | Sets a new description for the room | yes | yes | no | no | NULL` |
-| `--public` | Modifies the room to be a public room (value 1) or private room (value 0) | yes | yes | no | no | NULL` |
-| `--readonly` | Modifies the room to be read-only (value 1) or read-write (value 0) | yes | yes | no | no | NULL` |
-| `--listable` | Modifies the room's listable scope | yes | yes | no | no | NULL` |
-| `--password` | Sets a new password for the room; pass an empty value to remove password protection | yes | yes | no | no | NULL` |
-| `--owner` | Sets the given user as owner of the room; pass an empty value to remove the owner | yes | yes | no | no | NULL` |
-| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | no | NULL` |
+| `--name` | Sets a new name for the room | yes | yes | no | NULL` |
+| `--description` | Sets a new description for the room | yes | yes | no | NULL` |
+| `--public` | Modifies the room to be a public room (value 1) or private room (value 0) | yes | yes | no | NULL` |
+| `--readonly` | Modifies the room to be read-only (value 1) or read-write (value 0) | yes | yes | no | NULL` |
+| `--listable` | Modifies the room's listable scope | yes | yes | no | NULL` |
+| `--password` | Sets a new password for the room; pass an empty value to remove password protection | yes | yes | no | NULL` |
+| `--owner` | Sets the given user as owner of the room; pass an empty value to remove the owner | yes | yes | no | NULL` |
+| `--message-expiration` | Seconds to expire a message after sent. If zero will disable the expire message duration. | yes | yes | no | NULL` |
 
 ## talk:signaling:add
 
@@ -267,9 +267,9 @@ Add an external signaling server.
 | `server` | A server string, ex. wss://signaling.example.org | yes | no | `NULL` |
 | `secret` | A shared secret string. | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--verify` | Validate SSL certificate if set. | no | no | no | no | false` |
+| `--verify` | Validate SSL certificate if set. | no | no | no | false` |
 
 ## talk:signaling:delete
 
@@ -292,9 +292,9 @@ List external signaling servers.
 
 * `talk:signaling:list [--output [OUTPUT]]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:stun:add
 
@@ -330,9 +330,9 @@ List STUN servers.
 
 * `talk:stun:list [--output [OUTPUT]]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:turn:add
 
@@ -349,10 +349,10 @@ Add a TURN server.
 | `server` | A domain name, ex. turn.nextcloud.com | yes | no | `NULL` |
 | `protocols` | Protocols, can be udp or tcp or udp,tcp. | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--secret` | A shard secret string | yes | yes | no | no | NULL` |
-| `--generate-secret` | Generate secret if set. | no | no | no | no | false` |
+| `--secret` | A shard secret string | yes | yes | no | NULL` |
+| `--generate-secret` | Generate secret if set. | no | no | no | false` |
 
 ## talk:turn:delete
 
@@ -377,9 +377,9 @@ List TURN servers.
 
 * `talk:turn:list [--output [OUTPUT]]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | no | 'plain'` |
+| `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:user:remove
 
@@ -389,9 +389,9 @@ Remove a user from all their rooms
 
 * `talk:user:remove [--user USER]`
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--user` | Remove the given users from all rooms | yes | yes | yes | no | array ()` |
+| `--user` | Remove the given users from all rooms | yes | yes | yes | array ()` |
 
 ## talk:user:transfer-ownership
 
@@ -407,7 +407,7 @@ Adds the destination-user with the same participant type to all (not one-to-one)
 | `source-user` | Owner of conversations which shall be moved | yes | no | `NULL` |
 | `destination-user` | User who will be the new owner of the conversations | yes | no | `NULL` |
 
-| Options | Accept value | Is value required | Is multiple | is nagatable | Default |
+| Options | Accept value | Is value required | Is multiple | Default |
 |---|---|---|---|---|---|
-| `--include-non-moderator` | Also include conversations where the source-user is a normal user | no | no | no | no | false` |
-| `--remove-source-user` | Remove the source-user from the conversations | no | no | no | no | false` |
+| `--include-non-moderator` | Also include conversations where the source-user is a normal user | no | no | no | false` |
+| `--remove-source-user` | Remove the source-user from the conversations | no | no | no | false` |
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index 4762e73f379..9c9d7c22522 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -135,7 +135,7 @@ protected function describeInputDefinition(Command $command): string {
 				$text .= "\n\n";
 			}
 
-			$text .= "| Options | Accept value | Is value required | Is multiple | is nagatable | Default |\n";
+			$text .= "| Options | Accept value | Is value required | Is multiple | Default |\n";
 			$text .= '|---|---|---|---|---|---|';
 			foreach ($definition->getOptions() as $option) {
 				$text .= "\n";
@@ -160,9 +160,6 @@ protected function describeInputArgument(InputArgument $argument): string {
 
 	protected function describeInputOption(InputOption $option): string {
 		$name = '--'.$option->getName();
-		if ($option->isNegatable()) {
-			$name .= '|--no-'.$option->getName();
-		}
 		if ($option->getShortcut()) {
 			$name .= '|-'.str_replace('|', '|-', $option->getShortcut()).'';
 		}
@@ -174,7 +171,6 @@ protected function describeInputOption(InputOption $option): string {
 			($option->acceptValue() ? 'yes' : 'no')." | " .
 			($option->isValueRequired() ? 'yes' : 'no')." | " .
 			($option->isArray() ? 'yes' : 'no')." | " .
-			($option->isNegatable() ? 'yes' : 'no')." | " .
 			str_replace("\n", '', var_export($option->getDefault(), true)).'` |';
 	}
 }

From 011c87940d03fc03030aa167240c1abeed3c3fe4 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 22:56:43 -0300
Subject: [PATCH 08/12] Rename the GitHub Action

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 .github/workflows/occ-command-documentation.yml | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/occ-command-documentation.yml b/.github/workflows/occ-command-documentation.yml
index 9f439a25ad5..b6e6531914c 100644
--- a/.github/workflows/occ-command-documentation.yml
+++ b/.github/workflows/occ-command-documentation.yml
@@ -1,4 +1,4 @@
-name: Occ command documentation
+name: Documentation
 
 on:
   pull_request:
@@ -24,11 +24,11 @@ permissions:
   contents: read
 
 concurrency:
-  group: occ-command-documentation-${{ github.head_ref || github.run_id }}
+  group: occ-commands-${{ github.head_ref || github.run_id }}
   cancel-in-progress: true
 
 jobs:
-  occ-command-documentation:
+  occ-commands:
     runs-on: ubuntu-latest
 
     strategy:
@@ -88,4 +88,5 @@ jobs:
         id: check_documentation
         run: |
           ./occ talk:developer:update-docs
-          bash -c "[[ ! \"`git status --porcelain `\" ]] || (echo 'Documentation outdated. Run the command talk:developer:update-docs locally, commit the occ.md file and push the changes.' && exit 1)"
+          cd apps/${{ env.APP_NAME }}
+          bash -c "[[ ! \"`git status --porcelain `\" ]] || (cat docs/occ.md && echo 'Documentation outdated. Run the command talk:developer:update-docs locally, commit the occ.md file and push the changes.' && exit 1)"

From ff6a7eb4e50eb76d2f88291119a0100756c6d456 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 23:31:47 -0300
Subject: [PATCH 09/12] Removed unused option

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          |  6 ++----
 lib/Command/Developer/UpdateDocs.php | 28 +++++-----------------------
 2 files changed, 7 insertions(+), 27 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index ad1221e5e41..32462d04c87 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -81,12 +81,9 @@ Update documentation of commands
 
 ### Usage
 
-* `talk:developer:update-docs [<name>]`
+* `talk:developer:update-docs`
 
 
-| Arguments | Description | Is required | Is array | Default |
-|---|---|---|---|---|
-| `name` | The name of command to update. When have this argument, don't will upate the documentation, only will print the markdown output. | no | no | `NULL` |
 
 ## talk:monitor:calls
 
@@ -411,3 +408,4 @@ Adds the destination-user with the same participant type to all (not one-to-one)
 |---|---|---|---|---|---|
 | `--include-non-moderator` | Also include conversations where the source-user is a normal user | no | no | no | false` |
 | `--remove-source-user` | Remove the source-user from the conversations | no | no | no | false` |
+
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index 9c9d7c22522..b63fd5aba5b 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -35,7 +35,6 @@
 class UpdateDocs extends Base {
 	private IConfig $config;
 	private IAppManager $appManager;
-	private array $sections = [];
 
 	public function __construct(IConfig $config) {
 		$this->config = $config;
@@ -51,11 +50,6 @@ protected function configure(): void {
 		$this
 			->setName('talk:developer:update-docs')
 			->setDescription('Update documentation of commands')
-			->addArgument(
-				'name',
-				InputArgument::OPTIONAL,
-				"The name of command to update. When have this argument, don't will upate the documentation, only will print the markdown output."
-			)
 		;
 	}
 
@@ -63,27 +57,15 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		$this->appManager = \OC::$server->get(IAppManager::class);
 
 		$info = $this->appManager->getAppInfo('spreed');
-		$commandName = $input->getArgument('name');
-		$documentation = '';
+		$documentation = "# Talk occ commands\n\n";
 		foreach ($info['commands'] as $namespace) {
 			$command = $this->getCommand($namespace);
-			if ($commandName && $commandName !== $command->getName()) {
-				continue;
-			}
-			$this->sections['documentation'][] = $this->getDocumentation($command);
+			$documentation .= $this->getDocumentation($command) . "\n";
 		}
 
-		if ($commandName) {
-			$output->writeln(implode("\n", $this->sections['documentation']));
-		} else {
-			$documentation =
-				"# Talk occ commands\n\n" .
-				implode("\n", $this->sections['documentation']);
-
-			$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
-			fwrite($handle, $documentation);
-			fclose($handle);
-		}
+		$handle = fopen(__DIR__ . '/../../../docs/occ.md', 'w');
+		fwrite($handle, $documentation);
+		fclose($handle);
 		return 0;
 	}
 

From ad42bfb7f0559496a02a659eb1dccf77e974303d Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 2 Mar 2023 23:55:11 -0300
Subject: [PATCH 10/12] Clean code and small improvements at markdown

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 docs/occ.md                          | 51 ++++++++--------------------
 lib/Command/Developer/UpdateDocs.php | 13 ++++---
 2 files changed, 20 insertions(+), 44 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index 32462d04c87..dee27d24762 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -8,7 +8,6 @@ Add a new command
 
 * `talk:command:add <cmd> <name> <script> <response> <enabled>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `cmd` | The command as used in the chat "/help" => "help" | yes | no | `NULL` |
@@ -25,8 +24,6 @@ Adds some sample commands: /wiki, …
 
 * `talk:command:add-samples`
 
-
-
 ## talk:command:delete
 
 Remove an existing command
@@ -35,7 +32,6 @@ Remove an existing command
 
 * `talk:command:delete <command-id>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `command-id` |  | yes | no | `NULL` |
@@ -48,13 +44,12 @@ List all available commands
 
 * `talk:command:list [--output [OUTPUT]] [--] [<app>]`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `app` | Only list the commands of a specific app, "custom" to list all custom commands | no | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:command:update
@@ -65,7 +60,6 @@ Add a new command
 
 * `talk:command:update <command-id> <cmd> <name> <script> <response> <enabled>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `command-id` |  | yes | no | `NULL` |
@@ -83,8 +77,6 @@ Update documentation of commands
 
 * `talk:developer:update-docs`
 
-
-
 ## talk:monitor:calls
 
 Prints a list with conversations that have an active call as well as their participant count
@@ -94,7 +86,7 @@ Prints a list with conversations that have an active call as well as their parti
 * `talk:monitor:calls [--output [OUTPUT]]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:active-calls
@@ -106,7 +98,7 @@ Allows you to check if calls are currently in process
 * `talk:active-calls [--output [OUTPUT]]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:monitor:room
@@ -117,13 +109,12 @@ Prints a list with conversations that have an active call as well as their parti
 
 * `talk:monitor:room [--output [OUTPUT]] [--separator SEPARATOR] [--] <token>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room to monitor | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 | `--separator` | Separator for the CSV list when output=csv is used | yes | yes | no | ','` |
 
@@ -135,13 +126,12 @@ Adds users to a room
 
 * `talk:room:add [--user USER] [--group GROUP] [--] <token>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room to add users to | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--user` | Invites the given users to the room | yes | yes | yes | array ()` |
 | `--group` | Invites all members of the given groups to the room | yes | yes | yes | array ()` |
 
@@ -153,13 +143,12 @@ Create a new room
 
 * `talk:room:create [--description DESCRIPTION] [--user USER] [--group GROUP] [--public] [--readonly] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--moderator MODERATOR] [--message-expiration MESSAGE-EXPIRATION] [--] <name>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `name` | The name of the room to create | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--description` | The description of the room to create | yes | yes | no | NULL` |
 | `--user` | Invites the given users to the room to create | yes | yes | yes | array ()` |
 | `--group` | Invites all members of the given group to the room to create | yes | yes | yes | array ()` |
@@ -179,7 +168,6 @@ Deletes a room
 
 * `talk:room:delete <token>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room to delete | yes | no | `NULL` |
@@ -192,7 +180,6 @@ Demotes participants of a room to regular users
 
 * `talk:room:demote <token> <participant>...`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room in which users should be demoted | yes | no | `NULL` |
@@ -206,7 +193,6 @@ Promotes participants of a room to moderators
 
 * `talk:room:promote <token> <participant>...`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room in which users should be promoted | yes | no | `NULL` |
@@ -220,7 +206,6 @@ Remove users from a room
 
 * `talk:room:remove <token> <participant>...`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | Token of the room to remove users from | yes | no | `NULL` |
@@ -234,13 +219,12 @@ Updates a room
 
 * `talk:room:update [--name NAME] [--description DESCRIPTION] [--public PUBLIC] [--readonly READONLY] [--listable LISTABLE] [--password PASSWORD] [--owner OWNER] [--message-expiration MESSAGE-EXPIRATION] [--] <token>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `token` | The token of the room to update | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--name` | Sets a new name for the room | yes | yes | no | NULL` |
 | `--description` | Sets a new description for the room | yes | yes | no | NULL` |
 | `--public` | Modifies the room to be a public room (value 1) or private room (value 0) | yes | yes | no | NULL` |
@@ -258,14 +242,13 @@ Add an external signaling server.
 
 * `talk:signaling:add [--verify] [--] <server> <secret>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `server` | A server string, ex. wss://signaling.example.org | yes | no | `NULL` |
 | `secret` | A shared secret string. | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--verify` | Validate SSL certificate if set. | no | no | no | false` |
 
 ## talk:signaling:delete
@@ -276,7 +259,6 @@ Remove an existing signaling server.
 
 * `talk:signaling:delete <server>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `server` | An external signaling server string, ex. wss://signaling.example.org | yes | no | `NULL` |
@@ -290,7 +272,7 @@ List external signaling servers.
 * `talk:signaling:list [--output [OUTPUT]]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:stun:add
@@ -301,7 +283,6 @@ Add a new STUN server.
 
 * `talk:stun:add <server>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `server` | A domain name and port number separated by the colons, ex. stun.nextcloud.com:443 | yes | no | `NULL` |
@@ -314,7 +295,6 @@ Remove an existing STUN server.
 
 * `talk:stun:delete <server>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `server` | A domain name and port number separated by the colons, ex. stun.nextcloud.com:443 | yes | no | `NULL` |
@@ -328,7 +308,7 @@ List STUN servers.
 * `talk:stun:list [--output [OUTPUT]]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:turn:add
@@ -339,7 +319,6 @@ Add a TURN server.
 
 * `talk:turn:add [--secret SECRET] [--generate-secret] [--] <schemes> <server> <protocols>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `schemes` | Schemes, can be turn or turns or turn,turns. | yes | no | `NULL` |
@@ -347,7 +326,7 @@ Add a TURN server.
 | `protocols` | Protocols, can be udp or tcp or udp,tcp. | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--secret` | A shard secret string | yes | yes | no | NULL` |
 | `--generate-secret` | Generate secret if set. | no | no | no | false` |
 
@@ -359,7 +338,6 @@ Remove an existing TURN server.
 
 * `talk:turn:delete <schemes> <server> <protocols>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `schemes` | Schemes, can be turn or turns or turn,turns | yes | no | `NULL` |
@@ -375,7 +353,7 @@ List TURN servers.
 * `talk:turn:list [--output [OUTPUT]]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--output` | Output format (plain, json or json_pretty, default is plain) | yes | no | no | 'plain'` |
 
 ## talk:user:remove
@@ -387,7 +365,7 @@ Remove a user from all their rooms
 * `talk:user:remove [--user USER]`
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--user` | Remove the given users from all rooms | yes | yes | yes | array ()` |
 
 ## talk:user:transfer-ownership
@@ -398,14 +376,13 @@ Adds the destination-user with the same participant type to all (not one-to-one)
 
 * `talk:user:transfer-ownership [--include-non-moderator] [--remove-source-user] [--] <source-user> <destination-user>`
 
-
 | Arguments | Description | Is required | Is array | Default |
 |---|---|---|---|---|
 | `source-user` | Owner of conversations which shall be moved | yes | no | `NULL` |
 | `destination-user` | User who will be the new owner of the conversations | yes | no | `NULL` |
 
 | Options | Accept value | Is value required | Is multiple | Default |
-|---|---|---|---|---|---|
+|---|---|---|---|---|
 | `--include-non-moderator` | Also include conversations where the source-user is a normal user | no | no | no | false` |
 | `--remove-source-user` | Remove the source-user from the conversations | no | no | no | false` |
 
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index b63fd5aba5b..e467e35ce85 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -90,9 +90,8 @@ protected function getDocumentation(Command $command): string {
 				function ($carry, $usage) {
 					return $carry.'* `'.$usage.'`'."\n";
 				}
-			) . "\n";
+			);
 		$doc .= $this->describeInputDefinition($command);
-		$doc .= "\n";
 
 		return $doc;
 	}
@@ -100,7 +99,7 @@ function ($carry, $usage) {
 	protected function describeInputDefinition(Command $command): string {
 		$definition = $command->getDefinition();
 		$text = '';
-		if ($showArguments = \count($definition->getArguments()) > 0) {
+		if (\count($definition->getArguments()) > 0) {
 			$text .= "\n";
 			$text .= "| Arguments | Description | Is required | Is array | Default |\n";
 			$text .= '|---|---|---|---|---|';
@@ -110,21 +109,21 @@ protected function describeInputDefinition(Command $command): string {
 					$text .= $describeInputArgument;
 				}
 			}
+			$text .= "\n";
 		}
 
 		if (\count($definition->getOptions()) > 0) {
-			if ($showArguments) {
-				$text .= "\n\n";
-			}
+			$text .= "\n";
 
 			$text .= "| Options | Accept value | Is value required | Is multiple | Default |\n";
-			$text .= '|---|---|---|---|---|---|';
+			$text .= '|---|---|---|---|---|';
 			foreach ($definition->getOptions() as $option) {
 				$text .= "\n";
 				if (null !== $describeInputOption = $this->describeInputOption($option)) {
 					$text .= $describeInputOption;
 				}
 			}
+			$text .= "\n";
 		}
 		return $text;
 	}

From 3f93defc5c2df80937ba80a996be6852fee83f00 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Fri, 3 Mar 2023 07:26:49 +0100
Subject: [PATCH 11/12] fix: skip this command

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 docs/occ.md                          | 8 --------
 lib/Command/Developer/UpdateDocs.php | 4 ++++
 2 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/docs/occ.md b/docs/occ.md
index dee27d24762..cbd820e15f3 100644
--- a/docs/occ.md
+++ b/docs/occ.md
@@ -69,14 +69,6 @@ Add a new command
 | `response` | Who should see the response: 0 - No one, 1 - User, 2 - All | yes | no | `NULL` |
 | `enabled` | Who can use this command: 0 - Disabled, 1 - Moderators, 2 - Users, 3 - Guests | yes | no | `NULL` |
 
-## talk:developer:update-docs
-
-Update documentation of commands
-
-### Usage
-
-* `talk:developer:update-docs`
-
 ## talk:monitor:calls
 
 Prints a list with conversations that have an active call as well as their participant count
diff --git a/lib/Command/Developer/UpdateDocs.php b/lib/Command/Developer/UpdateDocs.php
index e467e35ce85..b6f05aac2c4 100644
--- a/lib/Command/Developer/UpdateDocs.php
+++ b/lib/Command/Developer/UpdateDocs.php
@@ -59,6 +59,10 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		$info = $this->appManager->getAppInfo('spreed');
 		$documentation = "# Talk occ commands\n\n";
 		foreach ($info['commands'] as $namespace) {
+			if ($namespace === self::class) {
+				continue;
+			}
+
 			$command = $this->getCommand($namespace);
 			$documentation .= $this->getDocumentation($command) . "\n";
 		}

From 7f6dbab1a087b798fbbe83c2e11c06bdc448b222 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Fri, 3 Mar 2023 07:36:43 +0100
Subject: [PATCH 12/12] Reorganize more documentation pages

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 docs/TURN.md                |  2 ++
 docs/capabilities.md        |  4 +---
 docs/constants.md           |  4 +---
 docs/system-requirements.md | 19 -------------------
 docs/user-requirements.md   | 20 ++++++++++++++++++++
 mkdocs.yml                  | 19 +++++++++++--------
 6 files changed, 35 insertions(+), 33 deletions(-)
 create mode 100644 docs/user-requirements.md

diff --git a/docs/TURN.md b/docs/TURN.md
index 01b0e9b278b..1bb474633ba 100644
--- a/docs/TURN.md
+++ b/docs/TURN.md
@@ -1,3 +1,5 @@
+# CoTURN configuration
+
 ### Background
 The configuration of Nextcloud Talk mainly depends on your desired usage:
 
diff --git a/docs/capabilities.md b/docs/capabilities.md
index a8400287165..1c1b52221a8 100644
--- a/docs/capabilities.md
+++ b/docs/capabilities.md
@@ -1,6 +1,4 @@
----
-title: Capabilities
----
+# Capabilities
 
 ## 3.0 (Initial Talk release)
 * `audio` - audio is supported
diff --git a/docs/constants.md b/docs/constants.md
index e30c13b00c8..340eba1f9cd 100644
--- a/docs/constants.md
+++ b/docs/constants.md
@@ -1,6 +1,4 @@
----
-title: Constants
----
+# Constants
 
 ## Conversation
 
diff --git a/docs/system-requirements.md b/docs/system-requirements.md
index c0ec8d34336..aa734991837 100644
--- a/docs/system-requirements.md
+++ b/docs/system-requirements.md
@@ -37,22 +37,3 @@ In Apache, if _mod_mime_ and _.htaccess_ files are enabled, the default _.htacce
 ## TURN server
 
 A TURN server running on **port 443** (or 80) is required in almost all scenarios, see  [Configuring coTURN](TURN.md) for more details.
-
-# Browsers
-
-## Recommended
-
-* Firefox: latest
-* Chrome/Chromium: latest
-
-## Supported
-
-* Firefox / Firefox ESR: 52 or later
-* Chrome / Chromium: 49 or later
-* Edge: latest
-* Safari: 13 or later
-
-# Mobile apps
-
-* Android: 6 or later
-* iOS: 14 or later
diff --git a/docs/user-requirements.md b/docs/user-requirements.md
new file mode 100644
index 00000000000..0c606897e54
--- /dev/null
+++ b/docs/user-requirements.md
@@ -0,0 +1,20 @@
+# User system requirements
+
+## Browsers
+
+### Recommended
+
+* Firefox: latest
+* Chrome/Chromium: latest
+
+### Supported
+
+* Firefox / Firefox ESR: 52 or later
+* Chrome / Chromium: 49 or later
+* Edge: latest
+* Safari: 13 or later
+
+## Mobile apps
+
+* Android: 6 or later
+* iOS: 14 or later
diff --git a/mkdocs.yml b/mkdocs.yml
index 3a3d4910e1a..80777b7af94 100644
--- a/mkdocs.yml
+++ b/mkdocs.yml
@@ -17,10 +17,18 @@ markdown_extensions:
 
 nav:
   - 'Nextcloud Talk API': 'index.md'
-  - 'System requirements': 'system-requirements.md'
   - 'Call experience': 'call-experience.md'
-  - Constants: 'constants.md'
-  - Capabilities: 'capabilities.md'
+  - 'User system requirements': 'user-requirements.md'
+  - 'Administration documentation':
+      - 'Server system requirements': 'system-requirements.md'
+      - 'Configuring coTURN': 'TURN.md'
+      - 'Occ commands': 'occ.md'
+      - 'Commands': 'commands.md'
+      - 'Matterbridge integration': 'matterbridge.md'
+  - 'Developer documentation':
+      - 'Constants': 'constants.md'
+      - 'Capabilities': 'capabilities.md'
+      - 'PHP events': 'events.md'
   - 'API documentation':
       - 'Conversations management': 'conversation.md'
       - 'Participants management': 'participant.md'
@@ -34,8 +42,3 @@ nav:
       - 'Settings': 'settings.md'
       - 'Integration by other apps': 'integration.md'
       - 'Signaling API': 'internal-signaling.md'
-  - Commands: 'commands.md'
-  - 'Occ commands': 'occ.md'
-  - 'PHP events': 'events.md'
-  - 'Configuring coTURN': 'TURN.md'
-  - 'Matterbridge integration': 'matterbridge.md'